Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 504674 - dev-ruby/storable-0.8.9-r2 - src_prepare(): The source directory '.../work/delano-storable-*' doesn't exist
Summary: dev-ruby/storable-0.8.9-r2 - src_prepare(): The source directory '.../work/d...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Gentoo Ruby Team
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2014-03-15 10:47 UTC by Joël Thieffry
Modified: 2014-03-25 06:35 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
dev-ruby/storable-0.8.9-r2.ebuild (storable-0.8.9-r2.ebuild,1.25 KB, text/plain)
2014-03-15 10:50 UTC, Joël Thieffry
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Joël Thieffry 2014-03-15 10:47:40 UTC
Hi !

This build fails because the directory inside the github archive contains the first digits of the SHA1 of the git branch, for example : delano-storable-c2bca6e. This leads to the following error :

>>> Unpacking storable-git-0.8.9.tgz to /home/var_tmp/portage/dev-ruby/storable-0.8.9-r1/work/all
>>> Source unpacked in /home/var_tmp/portage/dev-ruby/storable-0.8.9-r1/work
 * ERROR: dev-ruby/storable-0.8.9-r1::gentoo failed (prepare phase):
 *   The source directory '/home/var_tmp/portage/dev-ruby/storable-0.8.9-r1/work/delano-storable-*' doesn't exist

The proposed ebuild fixes that by :
 - in the src_unpack phase, renaming this directory to "all", which is the name expected by the next phase
 - unfixing the S variable :)

Best Regards,
Joël Thieffry.
Comment 1 Joël Thieffry 2014-03-15 10:50:13 UTC
Created attachment 372716 [details]
dev-ruby/storable-0.8.9-r2.ebuild
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2014-03-15 14:24:06 UTC
Comment on attachment 372716 [details]
dev-ruby/storable-0.8.9-r2.ebuild

--- storable-0.8.9-r1.ebuild    2014-03-10 17:15:31.000000000 +0100
+++ -   2014-03-15 15:23:25.295770607 +0100
@@ -24,7 +24,6 @@
 IUSE=""
 
 SRC_URI="http://github.com/delano/${PN}/tarball/v${PV} -> ${PN}-git-${PV}.tgz"
-S="${WORKDIR}/delano-${PN}-*"
 
 # technically, it could work without either; on the other hand, it
 # would break a bit of stuff.
@@ -36,6 +35,11 @@
 
 ruby_add_bdepend "test? ( dev-ruby/tryouts:2 )"
 
+src_unpack() {
+       unpack ${A}
+       mv delano-${PN}-* all
+}
+
 all_ruby_prepare() {
        mv bin examples || die
 }
Comment 3 Hans de Graaff gentoo-dev Security 2014-03-25 06:35:06 UTC
We have support for this mechanism in the ruby eclasses with the RUBY_S variable but it looks like this was not applied when the EAPI was bumped. Now fixed.