#1: Rpmlint's output isn't clean

chitlesh(SPECS)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/livecd-tools-001-1.i386.rpm
E: livecd-tools statically-linked-binary /usr/lib/livecd-creator/run-init
chitlesh(SPECS)[0]$rpmlint
/home/chitlesh/rpmbuild/RPMS/i386/livecd-tools-debuginfo-001-1.i386.rpm
E: livecd-tools-debuginfo empty-debuginfo-package

#2: Parallel build is missing

#3: BuildRoot tag should be fixed according to FE guidelines

It fails to build on x86_64:
    Nie znaleziono pliku: /var/tmp/livecd-tools-001-1-root/usr/lib64/livecd-
creator/mayflower
    Nie znaleziono pliku: /var/tmp/livecd-tools-001-1-root/usr/lib64/livecd-
creator/run-init
"Nie znaleziono pliku": "File not found"

Are any plains when it will be added to the repo?

In response to comment 1:
Re #1: That binary indeed needs to be statically linked because that is what is
needed when transitioning from initramfs to the rootfs. It's no different than nash.
Re #2: Please clarify; I'm not sure what "Parallel build" means
Re #3: Will look up BuildRoot tag in FE guidelines and fix.

In response to comment 2:
Will probably be ExclusiveArch: i386 initially; am planning to add x86-64
support real soon. And at some point also ppc support. It's just not trivial to
support other arches due to the very nature of 

Thanks a bunch for the comments so far.

(In reply to comment #3)
> Are any plains when it will be added to the repo?

This package is currently under review. It will be added to the repository as
soon as the review is complete. If you want it to reach the repository faster,
help review it. 

I can review this within the next day or 2 (if no one gets to it first in the 
meantime)

First pass:

MUST: package creates a useless -debuginfo package.  I'd recommend adding to
specfile something like:
# note: includes a single statically-linked, stripped binary, 
# -debuginfo is pointless here
%define debug_package   %{nil}

I'm doing a test run of 
livecd ... --package=fedora-livecd-desktop
If all goes well, we'll be close to package approval.

Looks good, test run succeeded.  APPROVED.  I'll trust you'll add the debuginfo
bit after import.

Okay, there have been a lot of changes to the livecd-tools so I think it's worth
getting the spec another look over before we actually import it.

http://people.redhat.com/~katzj/review/livecd-tools/livecd-tools.spec
http://people.redhat.com/~katzj/review/livecd-tools/livecd-tools-002-1.src.rpm

The big change is that it's now a noarch package.   rpmlint output:
[katzj@erebor livecd]$ rpmlint
/home/katzj/src/rpm/SRPMS/livecd-tools-002-1.src.rpm
/home/katzj/src/rpm/RPMS/noarch/livecd-tools-002-1.noarch.rpm
E: livecd-tools hardcoded-library-path in /usr/lib/livecd-creator
E: livecd-tools hardcoded-library-path in /usr/lib/livecd-creator/mayflower

These are correct now that we're noarch.

E: livecd-tools only-non-binary-in-usr-lib

Not a binary, but it needs to be executable.  We could conceivably move it to
/usr/share and chmod when we copy it into the live root if really needed, though

Good stuff (still), APPROVED.

When/if you get round-tuit:

* maybe(?)/SHOULD include a better URL, like
URL:  http://fedoraproject.org/wiki/FedoraLiveCD
until something better comes along.

New Package CVS Request
=======================
Package Name: livecd-tools
Short Description: Tools for building live CD's
Owners: davidz,katzj
Branches: devel
InitialCC:

Committed and built for devel

Package fails review guidelines. A bug must be filed for any use of ExcludeArch,
and placed on the appropriate ExcludeArch tracker bug for Extras.

Please file an appropriate bug, outlining what needs to be done to fix the
problem. Presumably it's as trivial as just making the output bootable with yaboot?

(In reply to comment #4)
> Will probably be ExclusiveArch: i386 initially; am planning to add x86-64
> support real soon. And at some point also ppc support. It's just not trivial to
> support other arches due to the very nature of 

That unfinished sentence sounds like a promising start to something that should
be noted in the ExcludeArch bug, pointing people at what needs to be done :)

Looks like all that needs to be done is make it bootable (and not require
syslinux). I can probably come up with a patch with the basic mechanisms, which
makes it work for _only_ PPC. Making it generically DTRT on each arch is going
to take more python-fu than I have. Do you want to do that part first, giving me
empty stubs of ppc functions to fill in?

(In reply to comment #14)

Support for other archs/machine types we (need to consider more than just the
arch) is of course welcome. But this bug is not the right place to discuss that;
that needs to happen on the livecd list

 http://www.redhat.com/mailman/listinfo/fedora-livecd-list

I have a Powerbook G4 12" and I'd be more than happy to help out testing the ppc
bits there...

(In reply to comment #15)
> (In reply to comment #14)
> 
> Support for other archs/machine types we (need to consider more than just the
> arch) is of course welcome. But this bug is not the right place to discuss 
> that

Indeed. Fedora package review says there MUST be a separate bug for the
existence of the ExcludeArch: in the specfile. And that bug must be on the
FE-ExcludeArch tracker for the architecture(s) which are excluded.

> I have a Powerbook G4 12" and I'd be more than happy to help out testing the ppc
> bits there...

The basics are in http://git.infradead.org/?p=users/dwmw2/livecd-ppc.git

Obviously it needs to be abstracted a little to work for _both_ ppc and x86.

We could also do with being able to install the 64-bit and 32-bit kernels at the
same time, so that we can boot on all hardware. It seems silly to make a 'ppc64
livecd' which has all the same userspace as the ppc32 version, but just a
different kernel.

What's in the git tree should work for both x86 and ppc now, although it still
needs a few things fixed for CHRP, and we need to work out how to include the
64-bit kernel. Seems rather unnecessary to have a whole separate ppc64 livecd
when only the kernel would be different.

Package Change Request
=======================
Package Name: livecd-tools
Short Description: Tools for building live CD's
Owners: sundaram
Branches: EL-5
InitialCC: katzj, markmc

cvs done.