Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 465176 - sys-devel/icecream-1.0.0 version bump request
Summary: sys-devel/icecream-1.0.0 version bump request
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Cluster Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-08 23:44 UTC by Oliver Freyermuth
Modified: 2013-04-14 08:24 UTC (History)
1 user (show)

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


Attachments
updated ebuild (icecream-1.0.0.ebuild,2.79 KB, text/plain)
2013-04-10 08:09 UTC, Oliver Jaksch
Details
unchanged but renamed patch (1.0.0-glibc2.16.patch,342 bytes, patch)
2013-04-10 08:10 UTC, Oliver Jaksch
Details | Diff
unchanged but renamed patch (1.0.0-icecream-conf.d-verbosity.patch,339 bytes, patch)
2013-04-10 08:10 UTC, Oliver Jaksch
Details | Diff
unchanged but renamed patch (symlinks.patch) (1.0.0-symlinks.patch,861 bytes, patch)
2013-04-10 08:11 UTC, Oliver Jaksch
Details | Diff
new patch to respect bug #418377 (icecc-create-env.patch) (1.0.0-icecc-create-env.patch,456 bytes, patch)
2013-04-10 08:12 UTC, Oliver Jaksch
Details | Diff
adapted runscript (icecream,1.70 KB, text/plain)
2013-04-10 08:13 UTC, Oliver Jaksch
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Oliver Freyermuth 2013-04-08 23:44:41 UTC
After a long time of silence from upstream, apparently development after the currently in-tree-version 0.9.7 restarted. 
See here: ftp://ftp.suse.com/pub/projects/icecream/ or in the github-repository linked on their homepage. 
A few days ago, a 'final' version 1.0.0 has been released. Compared to 0.9.7, the changelog promises fixes to compiler-plugins and clang support. 

I tested bumping the ebuild myself after commenting out the patches and can at least confirm it compiled and installed fine, but I guess the patches should be adapted to the new release. 

Reproducible: Always
Comment 1 Oliver Jaksch 2013-04-10 08:09:27 UTC
Created attachment 345086 [details]
updated ebuild

...and here it is. It's the first shot of this updated ebuild. Compiles fine and works in my cluster with some gentoo and arch machines (mixed x86/x86_64 environment; the most with icecream-0.97, the scheduler is icecream-1.0.0)
I rediffed some files and included the fix from bug #418377#c6 .

Please check and report.
Comment 2 Oliver Jaksch 2013-04-10 08:10:28 UTC
Created attachment 345088 [details, diff]
unchanged but renamed patch
Comment 3 Oliver Jaksch 2013-04-10 08:10:50 UTC
Created attachment 345090 [details, diff]
unchanged but renamed patch
Comment 4 Oliver Jaksch 2013-04-10 08:11:20 UTC
Created attachment 345092 [details, diff]
unchanged but renamed patch (symlinks.patch)
Comment 5 Oliver Jaksch 2013-04-10 08:12:46 UTC
Created attachment 345094 [details, diff]
new patch to respect bug #418377 (icecc-create-env.patch)
Comment 6 Oliver Jaksch 2013-04-10 08:13:50 UTC
Created attachment 345096 [details]
adapted runscript

location of logfiles changed in 1.0.0
Comment 7 Oliver Freyermuth 2013-04-11 01:16:08 UTC
In a smaller setup than yours (only gentoo x86_64 machines) I can confirm your ebuild+patches is working perfectly well. 

However, I realized the newly added clang-support to upstreams icecream is not yet present, because: 
- icecream-config has hardcoded "for a in gcc cc c++ g++". The original makefile adds "$CLANG_SYMLINK_WRAPPERS" which resolves to "clang clang++" if clang is present, maybe this can be added if an additional use-flag llvm is set? 
- icecc-create-env now supports --gcc and --clang, but these parameters are not yet used in our icecream-create-env-file. 

As my time is currently very limited, I sadly did not try adding the clang-symlinks manually yet.
Comment 8 Oliver Freyermuth 2013-04-11 09:43:42 UTC
I just tried adding clang{,++} symlinks and indeed environment was built and distributed, but was non-functioning, sadly the logfiles did not point out any problem, and ICECC-calls just returned broken-pipe messages. 

The interesting thing is that clang-support is done in a backward-compatible way: The environment is built with compilerwrappers called "g++" and "gcc" so older icecream-nodes which explicitly call gcc/g++ can also run clang. 

Other details I noted during upgrade of all nodes: 
- The logdir is not created by the ebuild, and if the logdir has the wrong owner, iceccd won't complain but just not write any log. Maybe the ebuild should explicitly create the logdir, with correct owner, and the runscript check permissions? 
- If /var/cache/icecream has wrong permissions, iceccd fails without any output / daemonizing, and the runscript only realizes during STOP that iceccd vanished silently. Again, a check of permissions should solve this easily.
Comment 9 Oliver Freyermuth 2013-04-14 01:55:30 UTC
It appears that a different ebuild for icecream-1.0.0 has been pushed into portage just today. 
It also install clang-symlinks and appears to work just fine. 
The env-checking tools have been dropped, and the path has been changed to /usr/libexec/icecc/bin. The installed runscript is called icecream-r2, which is still a bug I will file separately. 
My other notes are invalid for the new version, as it does not switch logdir.