[Bug 329291] Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: debootstrap - Bootstrap a basic Debian GNU/Linux system
Alias: debootstrap-review

https://bugzilla.redhat.com/show_bug.cgi?id=329291





------- Additional Comments From pertusus at free.fr  2007-10-12 11:36 EST -------
(In reply to comment #3)
> Patrice: Thanks for your comment. Please don't confuse unstable with innovative.
> If there are any things that work better in "unstable" that you miss in etch
> version, feel free to tell me, I'd gladly either backport the patch or move to
> unstable branch.

I don't confuse those, believe me. It is only a rule of thumb 
that works for most of the packages I have seen that are in 
debian and in fedora (in fedora devel). Otherwise said in
general stuff in debian stable is often too old for fedora
devel.

In the debootstrap I packaged there were configs for
more distros (I guess they are for ubuntu), that would be nice
to have.

> Also, I didn't notice that MAKEDEV does reside in /sbin as I include sbin also
> in my regular path. I also built in mock, which also seems to includ /sbin in
> PATH. I'll fix that do say either /sbin/MAKEDEV or /dev/MAKEDEV in next
> revision. Which one do you suggest?

/sbin/MAKEDEV
But what about the comment from Suse:
# the .deb is used to get the devices tarball as Debian
# may expect other devices as MAKEDEV generates... 

Also when I tried that, I got an error saying something along
'only root can create special devices'. But apparently this
is not the case now. Maybe because of the way you invoke it.

If your method really works, indeed it is better than my kludge.

I looked at the resulting tarballs, they are very different. But
it may be because mine is devices-std.tar.gz and yours is 
devices.tar.gz.

> When it comes to FHS, I've already considered that. Here's what FHS says:
> 
> > /usr/lib includes object files, libraries, and internal binaries that are not
> > intended to be executed directly by users or shell scripts. [22]
> 
> Only these two files don't comply: /usr/lib/debootstrap/arch and
> /usr/lib/debootstrap/devices.tar.gz. I've choosen to leave them in /usr/lib do
> avoid deviation from the upstream. If there are still strong objection against
> this I will move them to /usr/share instead.

The files in lib/debootstrap are not object files, libraries, or 
internal binaries, they are scripts functions, and script 
'configurations' and the devices tarball. All these are better 
in %_datadir. 

> Though I found no particular bits in your package that I'd find usable for this
> one, I'd be glad if you comaintained the package once it is approved.

No problem, I'd be happy to do that. I already have too much 
packages so I did not want to be the primary maintainer, but
no problem to be co-maintainer.



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.