Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 235753 - net-nntp/slrn-0.9.9 version bump
Summary: net-nntp/slrn-0.9.9 version bump
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gentoo Net-news project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-26 09:23 UTC by konsti
Modified: 2009-08-16 06:27 UTC (History)
1 user (show)

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


Attachments
slrn-0.9.9 ebuild (slrn-0.9.9.ebuild,951 bytes, text/plain)
2008-08-26 09:24 UTC, konsti
Details
A working net-nntp/slrn-0.9.9.ebuild (slrn-0.9.9.ebuild,800 bytes, text/plain)
2008-09-26 21:15 UTC, Marek Szuba (RETIRED)
Details
patch to enable cancel-locks and correcting uudeview configuration (slrn-0.9.9_p1.ebuild.patch,907 bytes, patch)
2009-08-16 06:26 UTC, Sebastian Lüttich
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description konsti 2008-08-26 09:23:50 UTC
slrn 0.9.9 was released, see attached ebuild.

Generally copied over the slrn-0.9.8.1_p1.ebuild but inserted stripping of MAKEOPTS=-jx to -j1, otherwise a race condition occurs in install process:

make[1]: Entering directory `/usr/src/tmp/portage/net-nntp/slrn-0.9.9/work/slrn-0.9.9/src'
/usr/src/tmp/portage/net-nntp/slrn-0.9.9/work/slrn-0.9.9/src/x86objs/chkslang slrn 20000
slrnpull created in /usr/src/tmp/portage/net-nntp/slrn-0.9.9/work/slrn-0.9.9/src/x86objs
../autoconf/mkinsdir.sh /usr/src/tmp/portage/net-nntp/slrn-0.9.9/image//usr/bin
/usr/bin/install -c  -m 755 -s /usr/src/tmp/portage/net-nntp/slrn-0.9.9/work/slrn-0.9.9/src/x86objs/slrn /usr/src/tmp/portage/net-nntp/slrn-0.9.9/image//usr/bin
/usr/bin/install: cannot create regular file `/usr/src/tmp/portage/net-nntp/slrn-0.9.9/image//usr/bin': No such file or directory
make[1]: *** [installbin] Error 1

This way, enjoy the new release!
Comment 1 konsti 2008-08-26 09:24:11 UTC
Created attachment 163799 [details]
slrn-0.9.9 ebuild
Comment 2 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2008-08-26 14:19:47 UTC
(In reply to comment #1)
> Created an attachment (id=163799) [edit]
> slrn-0.9.9 ebuild
> 

That is incorrect. You would want to do something like: emake -j1 in src_compile() to force a non-parallel build.
Comment 3 konsti 2008-08-26 15:37:49 UTC
I must admit that I agree, my MAKEOPTS line does only append -j1 to -j3, right? Actually I see this watching emerge with ps: "make -j3 -j1" is running. The funny thing is, this works. What also works without this line is 
MAKEOPTS="-j1" emerge slrn 
but not "emake -j1 || ..." with my MAKEOPTS omitted. The funny thing is, even then I see "make -j3 -j1" running while watching my processes with ps. But the same die message occurs. 

So for now I don't know how to cope with this otherwise.

The die message "`/usr/src/tmp/portage/net-nntp/slrn-0.9.9/image//usr/bin': No such file or directory" complains about the directory missing but after emerge died an ls on this directory is succesful, so I suspected a parallel make race condition.

MAKEOPTS=-j3 is what I ahve in my /etc/make.conf.
Comment 4 Marek Szuba (RETIRED) archtester gentoo-dev 2008-09-26 21:15:57 UTC
Created attachment 166550 [details]
A working net-nntp/slrn-0.9.9.ebuild

The problem was that the installation procedure in the Makefile has been split into several targets such as installdirs, installbins etc. - with file-installing targets not depending on the directory-creating one.

The proper solution to the problem would be to create such dependencies - but that I believe should preferably be handled by upstream. As far as ebuilds are concerned, the most simple solution to the problem is IMHO to call "make installdirs" explicitly before "make install" - that way no patches have to be applied to the sources.

Attached you will find my version of the ebuild, which in addition to introducing the aforementioned workaround cleans up some leftovers from previous versions not necessary with 0.9.9. It has been tested on amd64 and x86, as reflected by the content of KEYWORDS.

Hopefully everything will be okay this time and 0.9.9 will make its way into Portage soon, we had to wait long enough for a new version of slrn as it was :-)
Comment 5 Sven Wegener gentoo-dev 2008-10-07 21:32:01 UTC
I've commited 0.9.9_p1 to the tree.
Comment 6 konsti 2008-10-08 08:06:08 UTC
I installed it and it works fine :-)
Comment 7 Sebastian Lüttich 2009-08-16 06:26:59 UTC
Created attachment 201392 [details, diff]
patch to enable cancel-locks and correcting uudeview configuration

diff against the actual slrn-0.9.9_p1.ebuild

Enables cancel-lock headers and correcting --with-uudeview to --with-uu.

Sebastian