Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 213081 - version bump: net-firewall/conntrack-tools 0.9.6 released
Summary: version bump: net-firewall/conntrack-tools 0.9.6 released
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Cédric Krier
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-11 22:44 UTC by Stefan Behte (RETIRED)
Modified: 2008-03-20 04:16 UTC (History)
1 user (show)

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


Attachments
conntrack-tools-0.9.6.ebuild (conntrack-tools-0.9.6.ebuild,1.45 KB, text/plain)
2008-03-19 07:32 UTC, Peter Volkov (RETIRED)
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Behte (RETIRED) gentoo-dev Security 2008-03-11 22:44:13 UTC
See:
http://www.netfilter.org/projects/conntrack-tools/downloads.html
Comment 1 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 07:22:04 UTC
Jeroen when you bump ebuild, please, *read* INSTALL files and configure.in and update dependencies in ebuild. Also, please, comment what do you think about new kernel config options. Currently we have kernels from 2.6.16 up to 2.6.24. What kernel config options are required for which version. Thank you.
Comment 2 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 07:32:07 UTC
Created attachment 146548 [details]
conntrack-tools-0.9.6.ebuild

Also drop installation of .svn files. And check, you broke docs installation.


Actually I've fixed deps, docs in my overlay (ebuild, work in progress attached), but as you stepped forward and bumped, now I need you to read trough INSTALL file and do all durty work that should be done. There are new kernel config options but they are mentioned only for latest kernel (2.6.24) while we have 2.6.18-2.6.24 versions in our tree.  There are analogical options but with different names... So please, go and check in what options are required in older kernels.
Comment 3 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 07:51:29 UTC
Jaroen, also avoid patches for simple things like dropping -Werror. Use sed instead. One liner like

sed -i '/-Werror \\/d' {.,src,extensions}/Makefile.in

(not tested) should do the work much better and it's easier to maintain.
Comment 4 Cédric Krier gentoo-dev 2008-03-19 09:47:42 UTC
This version doesn't compile with gcc-3.4.6:
In file included from sync-alarm.c:21:
../include/network.h:64: warning: declaration of 'conf' shadows a global declaration
../include/conntrackd.h:144: warning: shadowed declaration is here
In file included from sync-mode.c:27:
../include/network.h:64: warning: declaration of 'conf' shadows a global declaration
../include/conntrackd.h:144: warning: shadowed declaration is here
make[1]: *** [sync-mode.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [sync-alarm.o] Error 1
mv -f .deps/cache.Tpo .deps/cache.Po
make[1]: Leaving directory `/var/tmp/portage/net-firewall/conntrack-tools-0.9.6/work/conntrack-tools-0.9.6/src'
make: *** [all-recursive] Error 1


This is already report to upstream.
That why I didn't yet bump the package.
Please before bump, ask me ?
Comment 5 Cédric Krier gentoo-dev 2008-03-19 09:59:30 UTC
I mask the ebuild until it will be fixed.
Comment 6 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 12:11:28 UTC
Cédric, are you sure that ebuild is the tree does not work? sed script I've posted in comment #3 (does the same as patch Jeroen did) works and fixes compilation problem. Also it's better to mask packages in profile/package.mask. What you did is clear indication for our arch teams that this package does not work on all archs... But it's not the issue here.


And again, kernel configuration options ebuild checks should be checked *before* bump.
Comment 7 Cédric Krier gentoo-dev 2008-03-19 13:07:28 UTC
Ok, fixed and unmask.
I think I was working on an old dir when testing the patch.
Comment 8 Jeroen Roovers (RETIRED) gentoo-dev 2008-03-19 16:10:59 UTC
I already masked this version before I even put it in the tree, I think.
Comment 9 Jeroen Roovers (RETIRED) gentoo-dev 2008-03-19 16:12:36 UTC
(In reply to comment #8)
> I already masked this version before I even put it in the tree, I think.

I sure did:

# Jeroen Roovers <jer@gentoo.org> (19 Mar 2008)
# Development version:
=net-firewall/conntrack-tools-0.9.6

Also, upstream says this is a development release (and doesn't recommend upgrading to it). It's bound to break on certain configurations and platforms.
Comment 10 Cédric Krier gentoo-dev 2008-03-19 16:41:36 UTC
(In reply to comment #9)
> 
> Also, upstream says this is a development release (and doesn't recommend
> upgrading to it). It's bound to break on certain configurations and platforms.
> 

See:
http://people.netfilter.org/pablo/conntrack-tools/news.html

"Upgrade is strongly recommended!"
So I will unmask it.
Comment 11 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 20:05:13 UTC
(In reply to comment #9)
> Also, upstream says this is a development release (and doesn't recommend
> upgrading to it). It's bound to break on certain configurations and platforms.

Sorry Jeroen, but have to CC you again. Taking into account that conntrack-tools are development it seems to me that masking of conntrack was too fast and it's better to unmask it. Missed homepage does not mean broken software or maybe it's even better to point homepage on conntrack-tools homepage?

Cédric, ebuild could be improved further: 1. there were changes in ebuild which change installed files - revision bump is necessary. 2. INSTALL file has different requirement on linux kernel then ebuild has. It should be linux kernel version >= 2.6.18. 3. take a look at this page:

http://people.netfilter.org/pablo/conntrack-tools/downloads.html

It contains different requirements on minimal net-libs/libnfnetlink version. What's correct version? I think forcing upper version is safer that's why I had it in my ebuild.

Reopening bug to take care of the above items. Thank you.
Comment 12 Cédric Krier gentoo-dev 2008-03-19 20:33:44 UTC
Fix in cvs, thanks
Comment 13 Peter Volkov (RETIRED) gentoo-dev 2008-03-19 20:50:12 UTC
Thank you, Cédric. I'm happy now.

But I still want to hear how Jeroen feels about unmasking conntrack too? See #11. Not reopening bug, since this bug is fixed, but I'm lazy to open new one for this discussing issue.
Comment 14 Jeroen Roovers (RETIRED) gentoo-dev 2008-03-20 04:16:50 UTC
I happen to be on the netmon alias, so no need to additionally CC me.

net-firewall/conntrack was merged into conntrack-tools quite a while ago, and both HOMEPAGE and SRC_URI are now defunct. I added to package.masks yesterday: one for conntrack and one for the conntrack-tools version bump that I felt wasn't perhaps up to standards yet. I guess that caused a lot of the confusion...