Summary: | sys-process/xjobs - reads job descriptions line by line and executes them in parallel | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Andrej Gelenberg <andrej.gelenberg> |
Component: | New packages | Assignee: | Default Assignee for New Packages <maintainer-wanted> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | CC: | mjo |
Priority: | Normal | Keywords: | EBUILD |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://www.maier-komor.de/xjobs.html | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
xjobs 20120412 ebuild
xjobs ebuild |
Description
Andrej Gelenberg
2012-05-30 07:58:53 UTC
Created attachment 313587 [details]
xjobs 20120412 ebuild
Comment on attachment 313587 [details]
xjobs 20120412 ebuild
DEPEND=""
RDEPEND=""
These can go, since they're empty anyway.
src_compile() {
This is exactly like the default src_compile for EAPI=0, so it should be removed too.
Speaking of EAPI, it should probably be set to 4, and then the || die() after dobin can go away too.
What is RESTRICT=mirror intended to do? It's GPL-2, so redistribution/mirroring should be freely allowed.
Created attachment 313629 [details]
xjobs ebuild
new try.
(In reply to comment #3) > Created attachment 313629 [details] > xjobs ebuild > > new try. The default src_install is no longer a no-op in EAPI4, so you can actually drop the whole thing, and portage will figure out what to do. Other minor things: * Copyright year should be updated * Description is a little too long * The IUSE variable is required, even if it's empty This would be an easy review, so I'd suggest submitting it to the sunrise[1] overlay. There is will be reviewed by a Gentoo developer, giving it a better chance of making it into the portage tree. [1] http://www.gentoo.org/proj/en/sunrise/ *xjobs-20140125 (19 Oct 2014) 19 Oct 2014; Michael Orlitzky <mjo@gentoo.org> +metadata.xml, +xjobs-20140125.ebuild: New package: sys-process/xjobs fixing bug #418211. Original ebuild contributed by Andrej Gelenberg. |