Bug 1211862

Summary:	u-boot misses patches in Factory
Product:	[SUSE ALP - SUSE Adaptable Linux Platform] Granite	Reporter:	Jiri Srain <jsrain>
Component:	Boot	Assignee:	Guilherme Moro <gmoro>
Status:	RESOLVED WORKSFORME	QA Contact:
Severity:	Normal
Priority:	P1 - Urgent	CC:	afaerber, ivan.ivanov, mbenes, mbrugger
Version:	unspecified
Target Milestone:	---
Hardware:	Other
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description Jiri Srain 2023-05-31 11:49:09 UTC

Submission to IBS did not get through, because of 

/work/src/done/SLE-ALP-SR-ST-1/u-boot was not checked in by dmach for the following reasons:
(submitrequest 300025 on https://build.suse.de)
dmach wrote (300025)
CONFIG_CMD_BTRFS=y is no longer set in couple places when comparing to SP5. Is this expected?
I ran: iosc rdiff {SUSE:SLE-15-SP5:GA,openSUSE.org:opensuse:Factory}/u-boot

Please, apply the missing patches to Factory; when submitting to Factory, please, submit at the same time to build.suse.de: SUSE:ALP:Source:Standard:1.0

Comment 1 Jiri Srain 2023-05-31 11:50:15 UTC

Be aware that it is essential for having aarch64 build for ALP Milestone1 next week.

Comment 2 Miroslav Beneš 2023-05-31 13:19:32 UTC

Matthias, could you a have look, please? My understanding is that you have maintained SLE's side of u-boot.

Comment 3 Matthias Brugger 2023-06-01 07:38:36 UTC

I have double checked and BTRFS is enabled. Actually I suppose there is a problem in the script because from my understanding
iosc rdiff {SUSE:SLE-15-SP5:GA,openSUSE.org:openSUSE:Factory}/u-boot

I only found entries like
> ^\ -CONFIG_CMD_BTRFS=y 
which is just context in a patch. I would expected something like
> ^+-CONFIG_CMD_BTRFS=y
Which would mean we are deleting line CONFIG_CMD_BTRFS=y in a patch in Factory.

I'm inclined to close this bug as worksforme

Comment 4 Ivan Ivanov 2023-07-18 10:52:03 UTC

Maybe is time to close this?

Comment 5 Matthias Brugger 2023-07-20 09:34:43 UTC

(In reply to Ivan Ivanov from comment #4)
> Maybe is time to close this?

Submit got accpeted as is.