New version out 5 days ago. I'll take care of bringing the ebuild up to date since we need to re-bake the perl patch. Just wanted to create a placeholder bug so others don't have to.
Created attachment 340900 [details] collectd-5.2.1.ebuild Here's an updated ebuild. Commitdiff here: https://github.com/jbergstroem/gentoo-overlay/compare/collectd-5.2.1 (I also removed the rm *.a line since there was no files to remove.)
+*collectd-5.2.1 (05 Mar 2013) + + 05 Mar 2013; Markos Chandras <hwoarang@gentoo.org> +collectd-5.2.1.ebuild: + Version bump. #454594. Thanks to Johan Bergström <bugs@bergstroem.nu> +
It seems like building without perl can bail in some setups (found this just now) during the configure phase. Sorry for not finding it in time, Markos. Here's a diff that fixes is: --- /usr/portage/app-admin/collectd/collectd-5.2.1.ebuild 2013-03-06 08:26:50.000000000 +1100 +++ collectd-5.2.1.ebuild 2013-03-06 14:20:26.728959748 +1100 @@ -289,7 +289,11 @@ fi # The perl bindings - myconf+=" $(use_with perl perl-bindings=INSTALLDIRS=vendor)" + if use perl; then + myconf+=" --with-perl-bindings=INSTALLDIRS=vendor" + else + myconf+=" --without-perl-bindings" + fi # Finally, run econf. KERNEL_DIR="${KERNEL_DIR}" econf --config-cache --without-included-ltdl $(use_enable static-libs static) --localstatedir=/var ${myconf} Do you think we should -r1?
Created attachment 341090 [details] build.log from building collectd without perl (In reply to comment #3) > Do you think we should -r1? Would be nice (at least update the ebuild) - I am affected by this problem: configure: error: invalid package name: perl-bindings=INSTALLDIRS=vendor
(In reply to comment #3) > It seems like building without perl can bail in some setups (found this just > now) during the configure phase. Sorry for not finding it in time, Markos. > > Here's a diff that fixes is: > --- /usr/portage/app-admin/collectd/collectd-5.2.1.ebuild 2013-03-06 > 08:26:50.000000000 +1100 > +++ collectd-5.2.1.ebuild 2013-03-06 14:20:26.728959748 +1100 > @@ -289,7 +289,11 @@ > fi > > # The perl bindings > - myconf+=" $(use_with perl perl-bindings=INSTALLDIRS=vendor)" > + if use perl; then > + myconf+=" --with-perl-bindings=INSTALLDIRS=vendor" > + else > + myconf+=" --without-perl-bindings" > + fi > > # Finally, run econf. > KERNEL_DIR="${KERNEL_DIR}" econf --config-cache --without-included-ltdl > $(use_enable static-libs static) --localstatedir=/var ${myconf} > > > Do you think we should -r1? No need for -r1. I presume those who hit this problem will not be able to configure/build the package. I will update the ebuild tonight.
+ 06 Mar 2013; Markos Chandras <hwoarang@gentoo.org> collectd-5.2.1.ebuild: + Fix building without perl +
