Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 1290136

Summary:	python-rospkg fails to rebuild on aarch64/ppc64le
Product:	[Fedora] Fedora	Reporter:	Yaakov Selkowitz <yselkowi>
Component:	python-rospkg	Assignee:	Rich Mattes <richmattes>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	logans, richmattes
Target Milestone:	---
Target Release:	---
Hardware:	aarch64
OS:	Linux
Whiteboard:
Fixed In Version:	python-rospkg-1.0.38-2.el7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-12-30 18:04:00 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	922257, 1285484

Description Yaakov Selkowitz 2015-12-09 17:43:51 UTC

Attempting to rebuild python-rospkg on an aarch64 machine fails due to the following test:

https://github.com/ros-infrastructure/rospkg/blob/master/test/test_rospkg_os_detect.py#L365

I'm not sure if that means only the listed architectures are supported, or if this is just a sanity test and other known architectures should be added to that list.

Comment 1 Rich Mattes 2015-12-10 00:22:22 UTC

It's just a sanity test.  I'm kind of confused that the package appears to have built successfully as recently as november on s390 and arm koji, I guess there are/were non-secondary arch builders that build noarch packages?  

Anyway, I'll make a patch and submit a pull request to add the additional arches that Fedora supports:

aarch64
ppc64le
s390
s390x

If you can think of any other uname -m strings let me know.

Comment 2 Yaakov Selkowitz 2015-12-10 01:05:58 UTC

(In reply to Rich Mattes from comment #1)
> It's just a sanity test.  I'm kind of confused that the package appears to
> have built successfully as recently as november on s390 and arm koji, I
> guess there are/were non-secondary arch builders that build noarch packages?

Currently, Fedora noarch packages are first built on primary koji (exactly which builder is luck-of-the-draw) then imported into secondary kojis without being rebuilt.  However, for EPEL 7, ppc64le is in the process of being added to the primary koji, with aarch64 to follow in 2016.  Therefore, in the very near future, epel7 noarch builds might end up built on one of the new arches, in which case a future build of this package will fail needlessly.  The same would apply to Fedora builds should any of these arches be moved to the primary koji.

> Anyway, I'll make a patch and submit a pull request to add the additional
> arches that Fedora supports:
> 
> aarch64
> ppc64le
> s390
> s390x
> 
> If you can think of any other uname -m strings let me know.

I think that covers it.  For the reasons stated above, please merge this into epel7 as well.

Comment 3 Scott K Logan 2015-12-10 01:24:21 UTC

I already had to patch this test once in upstream to support armv7hl. I think that was about 2 years ago, though. They were largely supportive of the change.

--scott

Comment 4 Rich Mattes 2015-12-10 02:48:51 UTC

Yes, I saw you in the git blame when I was trying to figure out when this test was added :)

I will apply the patch to epel 7 and rawhide, and let the rest of the branches be until the next upstream release.

Upstream pull request at:
https://github.com/ros-infrastructure/rospkg/pull/95

Comment 5 Fedora Update System 2015-12-10 03:11:16 UTC

python-rospkg-1.0.38-2.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-1e501c119f

Comment 6 Fedora Update System 2015-12-10 11:21:33 UTC

python-rospkg-1.0.38-2.el7 has been pushed to the Fedora EPEL 7 testing repository. If problems still persist, please make note of it in this bug report.
If you want to test the update, you can install it with
$ su -c 'yum --enablerepo=epel-testing update python-rospkg'
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-1e501c119f

Comment 7 Fedora Update System 2015-12-30 18:03:56 UTC

python-rospkg-1.0.38-2.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report.