You seem to forget your own requirements pretty often, hm? Portage 2.2.0_alpha120 (default/linux/amd64/10.0, gcc-4.7.1-asneeded, glibc-2.16.0, 3.4.4-hardened-r1 x86_64) ================================================================= System uname: Linux-3.4.4-hardened-r1-x86_64-AMD_Opteron-TM-_Processor_6272-with-gentoo-2.1 Timestamp of tree: Mon, 23 Jul 2012 14:30:01 +0000 ccache version 3.1.7 [disabled] app-shells/bash: 4.2_p37 dev-java/java-config: 2.1.12 dev-lang/python: 2.6.8, 2.7.3-r2, 3.2.3-r1 dev-util/ccache: 3.1.7 dev-util/cmake: 2.8.8-r3 sys-apps/baselayout: 2.1-r1 sys-apps/openrc: 0.10.5 sys-apps/sandbox: 2.6 sys-devel/autoconf: 2.13, 2.69 sys-devel/automake: 1.4_p6-r1, 1.9.6-r3, 1.10.3, 1.11.6, 1.12.2 sys-devel/binutils: 2.22-r1 sys-devel/gcc: 4.5.4, 4.6.3, 4.7.1 sys-devel/gcc-config: 1.7.3 sys-devel/libtool: 2.4.2 sys-devel/make: 3.82-r3 sys-kernel/linux-headers: 3.5 (virtual/os-headers) sys-libs/glibc: 2.16.0 Repositories: gentoo tbamd64 ACCEPT_KEYWORDS="amd64 ~amd64" ACCEPT_LICENSE="*" CBUILD="x86_64-pc-linux-gnu" CFLAGS="-O2 -pipe -ggdb -march=native -ftracer -frecord-gcc-switches" CHOST="x86_64-pc-linux-gnu" CONFIG_PROTECT="/etc /etc/entropy /opt/openjms/config /usr/share/config /usr/share/gnupg/qualified.txt /usr/share/maven-bin-3.0/conf /usr/share/openvpn/easy-rsa /usr/share/qpsmtpd/plugins /usr/share/themes/oxygen-gtk/gtk-2.0 /var/bind /var/lib/hsqldb /var/lib/redmine/config /var/spool/torque" CONFIG_PROTECT_MASK="${EPREFIX}/etc/gconf /etc/ca-certificates.conf /etc/dconf /etc/env.d /etc/fonts/fonts.conf /etc/games/angband/edit/ /etc/gconf /etc/gentoo-release /etc/php/apache2-php5.3/ext-active/ /etc/php/apache2-php5.4/ext-active/ /etc/php/cgi-php5.3/ext-active/ /etc/php/cgi-php5.4/ext-active/ /etc/php/cli-php5.3/ext-active/ /etc/php/cli-php5.4/ext-active/ /etc/revdep-rebuild /etc/sandbox.d /etc/splash /etc/terminfo /etc/texmf/language.dat.d /etc/texmf/language.def.d /etc/texmf/updmap.d /etc/texmf/web2c /var/lib/redmine/config/locales /var/lib/redmine/config/settings.yml" CXXFLAGS="-O2 -pipe -ggdb -march=native -ftracer -frecord-gcc-switches" DISTDIR="/var/cache/portage/distfiles" FCFLAGS="-O2 -pipe -ggdb -march=native -frecord-gcc-switches" FEATURES="assume-digests binpkg-logs config-protect-if-modified distlocks ebuild-locks fail-clean fixlafiles news parallel-fetch parse-eapi-ebuild-head protect-owned sandbox sfperms strict test test-fail-continue unknown-features-warn unmerge-orphans userfetch userpriv usersandbox" FFLAGS="-O2 -pipe -ggdb -march=native -frecord-gcc-switches" GENTOO_MIRRORS="http://ftp.ucsb.edu/pub/mirrors/linux/gentoo/ http://gentoo.mirrors.hoobly.com/ http://gentoo.llarian.net/" LANG="en_US.utf8" LC_ALL="C" LDFLAGS="-Wl,-O1 -Wl,--as-needed" MAKEOPTS="-j24" PKGDIR="/usr/portage/packages" PORTAGE_CONFIGROOT="/" PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --human-readable --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages" PORTAGE_TMPDIR="/tmp" PORTDIR="/var/cache/tinderbox/tree" PORTDIR_OVERLAY="/root/overlay" SYNC="rsync://rsync.gentoo.org/gentoo-portage" USE="3dnow 3dnowex acl amd64 berkdb bzip2 cli cracklib crypt cups cxx dri emacs ffmpeg fortran gdbm gnutls gphoto gphoto2 gpm iconv ipv6 mmx modules mudflap multilib ncurses nls nptl openmp pam pax_kernel pcre pdf plasma pppd qt3support readline semantic-desktop session sse sse2 sse3 sse4 ssl ssse3 tcmalloc tcpd udev unicode vhosts xorg zlib" ALSA_CARDS="ali5451 als4000 atiixp atiixp-modem bt87x ca0106 cmipci emu10k1x ens1370 ens1371 es1938 es1968 fm801 hda-intel intel8x0 intel8x0m maestro3 trident usb-audio via82xx via82xx-modem ymfpci" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mmap_emul mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache cgi cgid dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" CALLIGRA_FEATURES="kexi words flow plan sheets stage tables krita karbon braindump" CAMERAS="ptp2" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ubx" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LIBREOFFICE_EXTENSIONS="presenter-console presenter-minimizer" PHP_TARGETS="php5-3" PYTHON_TARGETS="python3_2 python2_7" RUBY_TARGETS="ruby18 ruby19 ree18" USERLAND="GNU" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account" Unset: CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LINGUAS, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS, USE_PYTHON
(In reply to comment #0) > You seem to forget your own requirements pretty often, hm? Find yourself a girlfriend or something.
How was this a bug?? ./configure doesn't use pkg-config, and AFAIK we don't need to have pkg-config as a dependency in order to install .pc files do we? Same way we don't need openrc to have /etc/init.d/ files installed, or have systemd for systemd unit files... I vote to reverse the commit and remove the dep...
(In reply to comment #2) > How was this a bug?? > > ./configure doesn't use pkg-config, and AFAIK we don't need to have > pkg-config as a dependency in order to install .pc files do we? Same way we > don't need openrc to have /etc/init.d/ files installed, or have systemd for > systemd unit files... > > I vote to reverse the commit and remove the dep... Automatic .la removal uses pkg-config to expand variables in .pc file to get provided library list.
... ok that one's new to me. I take it, it doesn't matter that this package doesn't and will never have .la files? :)
(In reply to comment #4) > ... ok that one's new to me. > > I take it, it doesn't matter that this package doesn't and will never have > .la files? :) Hmm, I guess that could be actually change for the pkg-config branch to run only when there's at least a single .la file for consideration.
It might also be worth considering to make autotools-utils.eclass (or eutils.eclass) add virtual/pkgconfig to DEPEND, since it's that eclass which is (indirectly) forcing the prune_libtool_files call in src_install.. It doesn't make sense to me to require it in each ebuild just because autotools-uitls.eclass is inherited.
(In reply to comment #6) > It might also be worth considering to make autotools-utils.eclass (or > eutils.eclass) add virtual/pkgconfig to DEPEND, since it's that eclass which > is (indirectly) forcing the prune_libtool_files call in src_install.. > > It doesn't make sense to me to require it in each ebuild just because > autotools-uitls.eclass is inherited. Well, it's necessary only if the package installs .pc files.
Hey it's not me who argued it wasn't a complete screw up. Guess somebody should think more about his mistakes rather than insulting those who report their mistakes.
(In reply to comment #8) > Hey it's not me who argued it wasn't a complete screw up. > > Guess somebody should think more about his mistakes rather than insulting > those who report their mistakes. Guess somebody should spend more time doing useful stuff rather than being an arrogant jerk.
That would be you, I assume.
(In reply to comment #10) > That would be you, I assume. Guess again. I'll give you a tip: it is a person who just complains believing that making people angry will actually increase their productivity and make all issues disappear.
Why instead of wasting people's time by fixing packages one by one you don't go back to _the main bug_ and start looking into fixing it? "Really, mister? I screwed up not following my own requirement, that's a fact." These are your words. "You seem to forget your own requirements pretty often, hm?" These are mine. Now before I start to be pissed off enough to ask somebody else to turn you down a notch, can you admit that the code committed to eutils eclass is messed up and that requirement shouldn't be there to begin with?
(In reply to comment #12) > Why instead of wasting people's time by fixing packages one by one you don't > go back to _the main bug_ and start looking into fixing it? Oh, sorry I wasted *your* time by committing the fix. I didn't know you are running CVS by hand, sorry. > Now before I start to be pissed off enough to ask somebody else to turn you > down a notch, can you admit that the code committed to eutils eclass is > messed up and that requirement shouldn't be there to begin with? Yes, the requirement is confusing. Noone catched the problem when the code was posted for review. Feel free to suggest a better solution. Until then, I believe that fixing the package to comply with the requirement is the correct solution.
