[Bug 249949] Review Request: beldi - Belug Linux Distribution Burner

[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: beldi - Belug Linux Distribution Burner


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





------- Additional Comments From fedora at christoph-wickert.de  2007-09-19 09:19 EST -------
(In reply to comment #5)
> Christoph, Beldi changed and uses now a directory in $HOME unless configured 
> somehow else.

Huh? Beldi changed? But the source is still named beldi-0.9.tar.gz. It's a bad
habit to change something without increasing the version. This version is
completely different from the previous one , so it should at least be called 0.9.1.

In fact I liked beldi in /var/lib. If we use userhelper multiple users could
share a single beldi installation, now every user needs to download all the ISOs
again, which is more than 20 GB per user!

By default beldi now uses /tmp, which is a bad idea, because
- now beldi complains that it can't read files already in /tmp
- it bears the risk that beldi changes files it can read/write
- /tmp will be wiped out on reboot and people will have to start "beldi
--update" and all the downloads again! 

Even worse: One can't change the download location because of
"Configuration::Save can't open file /etc/beldi_settings.xml in write mode."
So we still need root privileges or beldi needs to store the configuration in
~/.beldi_settings.xml.

> Oh and burning CDs on Fedora 7 works without special permissions for me.

Burning did not work for me with the previous release. There were permission
issues for normal users and as root there were problems with wodim, it was
probing the wrong devices (/dev/hd* instead of /dev/sd*).

With the new version I still see a lot of warnings but at least burning works.

> Core 6 can't be supported by Beldi right now, because of qemu requirement.

I'm not sure if beldi should require qemu because the "test with qemu" feature
doesn't work for most distributions I've tested. Ether not enough RAM or no
harddisk, but all the installers fail. This is only useful for those
distributions that come with an installable livecd.

This version of beldi looks so 'unfinished' to me (look at the configuration
page) that I only would allow it to enter rawhide but none of the stable
releases. In fact I noticed a regression: The downloader no longer seems to
accept "file://" as mirror url (I used local copies of the ISOs for testing).

I'm attaching my spec, sorry I did not put it into this review before. This is
how I would have packaged the previous version. Please do a diff against yours
and take what you think is useful.

-- 
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.