Bug 1178167 - Review Request: golang-github-smashwilson-gophercloud - The Go SDK for Openstack
Summary: Review Request: golang-github-smashwilson-gophercloud - The Go SDK for Openstack
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Lokesh Mandvekar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2015-01-02 16:16 UTC by Jan Chaloupka
Modified: 2015-01-05 11:07 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-01-05 11:07:16 UTC
Type: ---
Embargoed:
jchaloup: fedora-review?


Attachments (Terms of Use)

Description Jan Chaloupka 2015-01-02 16:16:43 UTC
Spec URL: https://jchaloup.fedorapeople.org/reviews/golang-github-smashwilson-gophercloud/golang-github-smashwilson-gophercloud.spec

SRPM URL: https://jchaloup.fedorapeople.org/reviews/golang-github-smashwilson-gophercloud/golang-github-smashwilson-gophercloud-0-0.1.gite13cda2.fc20.src.rpm

Description: The Go SDK for Openstack

Fedora Account System Username: jchaloup

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=8513961

$ rpmlint golang-github-smashwilson-gophercloud-0-0.1.gite13cda2.fc20.src.rpm golang-github-smashwilson-gophercloud-devel-0-0.1.gite13cda2.fc20.noarch.rpm golang-github-smashwilson-gophercloud.spec
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

Upstream: https://github.com/smashwilson/gophercloud

Info: has cyclic dependency on https://github.com/rackspace/gophercloud

Comment 1 Lokesh Mandvekar 2015-01-02 17:15:09 UTC
ohh man, smashwilson/gophercloud is a fork of rackspace/gophercloud, and those being cyclic deps is pretty insane, I guess worst case we could just install the sources and have them "Requires:" each other (Maybe one single main package could provide both rackspace/gophercloud and smashwilson/gophercloud to make this easier).

Perhaps better to check with upstream before we go ahead with this.

Comment 2 Jan Chaloupka 2015-01-05 11:07:16 UTC
Looking at files with github.com/smashwilson/gophercloud import, there are only two:

gophercloud-1.0.0/acceptance/rackspace/compute/v2/bootfromvolume_test.go:12
gophercloud-1.0.0/acceptance/openstack/compute/v2/bootfromvolume_test.go:11

Both are used only for testing. Thus this dependency is optional.


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