Bug#668100: fromiso rewriting based on findiso structure
adrian15 wrote (09 Apr 2012 18:43:54 GMT) :
> I attach the splitted patches.
* fromiso_004_fromiso_uses_is_mountpoint_function.patch *but*
I amended it to s,/root/live/fromiso,/live/fromiso, given I did
not apply the 003 patch
> I am doubting about 003 patch.
So am I.
> Why fromiso didn't have in the first place?
No idea. Why should it have it?
Given fromiso= works well without this change AFAIK,
I see no reason to clutter the code with it.
(No, I don't think making the code more similar to findiso= counts here.)
> Why findiso did it? Perhaps because fromiso in the paste did
> have it?
No idea, but dba started a discussion about it on #656135.
Try removing this piece of code from findiso= and see what happens?
Shall we consider this action item (fromiso rewriting based on findiso
structure) as done, and close this bug?
> Related to this I have found: /live/image and /root/live/image that
> do not seem to be used (just rgreped for /live/image). I might open
> a new bug for this but I'm not sure.
A new bug is needed if we want to go any further on this road.
| GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
| OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc