Home | Docs | Forums | Lists | Bugs | Planet | Store | GMN | Get Gentoo!
Not eligible to see or edit group visibility for this bug.
View Bug Activity | Format For Printing | XML | Clone This Bug
src_unpack() { unpack ${A} ; cd ${S} epatch ${FILESDIR}/${P}-setsid.patch ebegin "Trying to download the latest hddtemp.db file" wget -q --timeout=10 http://www.guzu.net/linux/hddtemp.db eend $? } wget should not be used in src_unpack. That db file should be in SRC_URI instead of in src_unpack, so that portage can fetch and digest it properly. This also defeats people who use emerge -f to fetch all important files, only to fail here with no internet connection.
i dont really think this is an issue ... the .db file is constantly updated upstream, the wget has a small timeout, and if it fails, there is a local (old) cache to fall back on
As Zac suggested on IRC, put the fetching in pkg_config and inform the user that they have to run it after installation to fetch the newer db.
The pkg_config idea sounds best to me if this is a moving target. Spock, any reason not to change it?
The only reason for not changing it earlier was that I've forgotten about this bug (since it wasn't assinged to me) :) A fixed ebuild is now in CVS. I didn't want to make any changes in the old ebuilds, so we'll have to wait for the new one to be marked stable so that -beta13 can be removed from the tree.