Bug 32816 - UPDATE REQUEST for mock-core-configs
Summary: UPDATE REQUEST for mock-core-configs
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 9
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard: MGA9-64-OK
Keywords: advisory, validated_update
Depends on:
Blocks:
 
Reported: 2024-02-08 11:57 CET by Philippe Didier
Modified: 2024-02-12 17:02 CET (History)
7 users (show)

See Also:
Source RPM: mock, mock-core-configs-39.1-1.mga9
CVE:
Status comment:


Attachments
Suggested advisory (405 bytes, text/plain)
2024-02-09 00:25 CET, katnatek
Details
Suggested advisory for the failure to build rpms for ARM arches bug (419 bytes, text/plain)
2024-02-09 16:06 CET, Len Lawrence
Details
Suggested advisory for the failure to build rpms for ARM arches bug (414 bytes, text/plain)
2024-02-09 16:15 CET, Len Lawrence
Details
Fixed advisory (403 bytes, text/plain)
2024-02-09 19:02 CET, katnatek
Details
Updated advisory (451 bytes, text/plain)
2024-02-10 04:01 CET, katnatek
Details

Description Philippe Didier 2024-02-08 11:57:23 CET
Since mock has been updated from 3.5 version to 5.1.1 in september it's no more possible to use mock to build rpms for arm arches (bug 32620) 

Fortunately Jani Välimaa found a way to correct this with a short patch

Upstream seems not wanting to correct this....  and this bug 32620 must be resolved

So here is an update with Jani's patch

ADVISORY
Mock couldn't build rpms for arm arches since it has been updated to 5.1.1 version
A little patch added to mock-core-configs allows now mock to work for all arches

RPM
mock-core-configs-39.1-2.mga9.noarch.rpm
from

SRPM
mock-core-configs-39.1-2.mga9.src.rpm

To test it :
1) before updating, try to build a rpm for aarch64 with mock
2) update
3) then try again to build the same rpm for aarch64
Philippe Didier 2024-02-08 11:57:45 CET

See Also: (none) => https://bugs.mageia.org/show_bug.cgi?id=32620

Philippe Didier 2024-02-08 11:59:19 CET

CC: (none) => geiger.david68210, j.alberto.vc, jani.valimaa

Comment 1 katnatek 2024-02-09 00:25:06 CET Comment hidden (obsolete)
katnatek 2024-02-09 00:26:19 CET

CC: (none) => marja11, tarazed25

Comment 2 katnatek 2024-02-09 00:27:20 CET
@Marja and Len, I still can't upload this myself, please do it for me
Comment 3 katnatek 2024-02-09 02:22:58 CET Comment hidden (obsolete)
katnatek 2024-02-09 02:23:19 CET

Whiteboard: (none) => MGA9-64-OK

Comment 4 Philippe Didier 2024-02-09 11:47:47 CET
For Information

This update to test has not been assigned to QA
and it doesn't appear for QA here :
http://madb.mageia.org/tools/updates


@katnatek
The attached suggested advisory contains a typo in the name :
mock-core-configs-39.1-2.mga9-1.0-1.mga9

instead of 
mock-core-configs-39.1-2.mga9.noarch.rpm
and
mock-core-configs-39.1-2.mga9.src.rpm
Comment 5 Len Lawrence 2024-02-09 16:06:58 CET
Created attachment 14348 [details]
Suggested advisory for the failure to build rpms for ARM arches bug

This is my take on this.  It may need reviewing by somebody more experienced.
Not obsoleting the previous version yet.
Comment 6 Len Lawrence 2024-02-09 16:15:32 CET
Created attachment 14349 [details]
Suggested advisory for the failure to build rpms for ARM arches bug

This is my take on this - it may require editing by more experienced hands.
Leaving the original in place.
Comment 7 Philippe Didier 2024-02-09 16:42:48 CET
Hi Lew

Unfortunately, your advisories have the same typo as katnatek's one :

mock-core-configs-39.1-2.mga9-1.0-1.mga9  (?)
instead of
mock-core-configs-39.1-2.mga9

Besides this there are still these problems :

This update to test has not been assigned to QA
and it doesn't appear for QA here :
http://madb.mageia.org/tools/updates
Comment 8 Len Lawrence 2024-02-09 16:54:43 CET
Sorry for not noticing the error in the package name - too much hurry and too much copy and paste maybe.
Comment 9 katnatek 2024-02-09 19:02:55 CET
Created attachment 14350 [details]
Fixed advisory

Use Len's advisory and fix the src.rpm

Attachment 14349 is obsolete: 0 => 1
Attachment 14347 is obsolete: 0 => 1
Attachment 14348 is obsolete: 0 => 1

katnatek 2024-02-09 19:05:39 CET

Assignee: bugsquad => qa-bugs

Comment 10 katnatek 2024-02-09 19:06:53 CET
Assigned to QA, note that I already QA the update in comment#3
Comment 11 Jani Välimaa 2024-02-09 20:48:15 CET
Please be patient, and don't rush at things. The patch added is not a proper fix, only a workaround.

Proper fix from upstream pull request [1] is available in Cauldron for mock and mock-core-configs. Both patches applies cleanly also to mga9 pkgs. I'll push an update soon for testing.

[1] https://github.com/rpm-software-management/mock/pull/1317

Assignee: qa-bugs => bugsquad
Whiteboard: MGA9-64-OK => (none)

Comment 12 Jani Välimaa 2024-02-09 21:09:43 CET
Patched mock and mock-core-configs pushed to mga9 core/updates_testing. Please test.

SRPMS:
mock-5.1.1-1.1.mga9
mock-core-configs-39.1-2.1.mga9

RPMS:
mock-5.1.1-1.1.mga9
mock-core-configs-39.1-2.1.mga9
mock-filesystem-5.1.1-1.1.mga9
mock-lvm-5.1.1-1.1.mga9
mock-scm-5.1.1-1.1.mga9

Assignee: bugsquad => qa-bugs

Comment 13 Philippe Didier 2024-02-10 02:45:57 CET
(In reply to Jani Välimaa from comment #11)
> Please be patient, and don't rush at things. The patch added is not a proper
> fix, only a workaround.
> 
> Proper fix from upstream pull request [1] is available in Cauldron for mock
> and mock-core-configs. Both patches applies cleanly also to mga9 pkgs. I'll
> push an update soon for testing.
> 
> [1] https://github.com/rpm-software-management/mock/pull/1317

Hi Jani
Thanks again for your perseverance
Sorry to not have been patient :

Indeed this comment from upstream 2 days ago :
"moved this from Needs triage to Someday in future in CPT Kanban "
made me think that this issue will not be taken in account before a long time !

Waiting this hypothetical correction from upstream  your patch (even it was a temporary workaround) allowed at least to use mock anew...

We missed this :
We need to test builds for arm arches at home and eventually modify the spec or add patches so that the build succeeds 
instead of blindly submitting srpms to the BS and fail to build.
That had not been possible for weeks inside a Mageia9 system


The good news is that finally upstream showed some interest to correct this

I have tested your updates  
It's OK for Mageia9 64bits : mock can build rpms for arm arches
Philippe Didier 2024-02-10 02:46:39 CET

Whiteboard: (none) => MGA9-64-OK

Comment 14 katnatek 2024-02-10 04:01:18 CET
Created attachment 14356 [details]
Updated advisory

Attachment 14350 is obsolete: 0 => 1

Comment 15 Thomas Andrews 2024-02-10 15:29:04 CET
Validating.

Keywords: (none) => validated_update
CC: (none) => andrewsfarm, sysadmin-bugs

Marja Van Waes 2024-02-10 21:18:38 CET

Source RPM: mock-core-configs-39.1-1.mga9.src.rpm => mock, mock-core-configs-39.1-1.mga9

Marja Van Waes 2024-02-10 21:23:21 CET

Keywords: (none) => advisory

Comment 16 Mageia Robot 2024-02-10 21:52:23 CET
An update for this issue has been pushed to the Mageia Updates repository.

https://advisories.mageia.org/MGAA-2024-0051.html

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 17 Philippe Didier 2024-02-12 15:22:16 CET
@katnatek
@jani

I don't know if this needs to reopen this bug ....

Today (12/02/2024 at 2pm UT) upstream has just committed a huge change inside the source to solve the https://github.com/rpm-software-management/mock/issues/1304 that was created there by Mageia :

https://github.com/rpm-software-management/mock/commit/f28e3cdf66c08d4a7e02732e9abb513552fa3d27
Comment 18 Jani Välimaa 2024-02-12 16:34:29 CET
Those are the changes I referred in comment 11, and applied to pkgs in comment 12.
Comment 19 Philippe Didier 2024-02-12 17:02:28 CET
(In reply to Jani Välimaa from comment #18)
> Those are the changes I referred in comment 11, and applied to pkgs in
> comment 12.

Hi Jani

Sorry for the noise
I have just seen the last commit in upstream Source... 
I had not understood that it was the same thing as your patch that had been merged

Thanks again for your contribution  to solve this bug for Mageia ... and for upstream !

Note You need to log in before you can comment on or make changes to this bug.