Created attachment 367918 [details, diff] patch against e2fsprogs-libs-1.42.9.ebuild Changes (trivial): - mint doesn't need a libtype - fix defines for uuid on mint
Comment on attachment 367918 [details, diff] patch against e2fsprogs-libs-1.42.9.ebuild >+ if [[ ${CHOST} == *-mint* ]]; then >+ sed -i -e 's/_SVID_SOURCE/_GNU_SOURCE/' lib/uuid/gen_uuid.c || die >+ fi this needs documentation. there's no details at all here. > case ${CHOST} in >- *-darwin*) libtype=bsd;; >- *) libtype=elf;; >+ *-darwin*) libtype=--enable-bsd-shlibs ;; >+ *-mint*) libtype= ;; >+ *) libtype=--enable-elf-shlibs ;; this is the point of the tc-is-static-only func. don't hardcode mint here.
(In reply to SpanKY from comment #1) > Comment on attachment 367918 [details, diff] [details, diff] > patch against e2fsprogs-libs-1.42.9.ebuild > > >+ if [[ ${CHOST} == *-mint* ]]; then > >+ sed -i -e 's/_SVID_SOURCE/_GNU_SOURCE/' lib/uuid/gen_uuid.c || die > >+ fi This part is no longer required and can be removed.
(In reply to SpanKY from comment #1) > this needs documentation. there's no details at all here. > > > case ${CHOST} in > >- *-darwin*) libtype=bsd;; > >- *) libtype=elf;; > >+ *-darwin*) libtype=--enable-bsd-shlibs ;; > >+ *-mint*) libtype= ;; > >+ *) libtype=--enable-elf-shlibs ;; > > this is the point of the tc-is-static-only func. don't hardcode mint here. @alan, can you comment on the use of tc-is-static-only? According to grobian the darwin part might not be need anymore, too.
It just needs this then.... $(tc-is-static-only || echo --enable-elf-shlibs) On the actual econf build line.
Created attachment 367964 [details, diff] patch against e2fsprogs-libs-1.42.9.ebuild (In reply to Alan Hourihane from comment #4) > It just needs this then.... > > $(tc-is-static-only || echo --enable-elf-shlibs) > > On the actual econf build line. Great, here we go.
Comment on attachment 367964 [details, diff] patch against e2fsprogs-libs-1.42.9.ebuild LGTM; feel free to commit
+ 17 Jan 2014; Christoph Junghans <ottxor@gentoo.org> + e2fsprogs-libs-1.42.9.ebuild: + added prefix support (bug #498212)