Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-26 Thread Tokarski Boleslaw
Hello, I just downloaded the latest git repository of choose-mirror and reviewed the code and found the second patch I applied against an older version of choose-mirror (2.29) by accident. So I looked at the correction of return code and found that this was no longer needed for 2.37 and

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-25 Thread Tokarski Boleslaw
Hello, Please test it and if it works ping us to review and commit it; I have tested the patch, found one bug, so I am attaching the updated patch. It worked in my environment, though I might have not considered all use cases. regarding debootstrap you can preseed it using: d-i

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-25 Thread Otavio Salvador
On Mon, Jul 25, 2011 at 15:30, Tokarski Boleslaw boleslaw.tokar...@tieto.com wrote: Hello, Please test it and if it works ping us to review and commit it; I have tested the patch, found one bug, so I am attaching the updated patch. It worked in my environment, though I might have not

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-22 Thread Tokarski Boleslaw
Package: choose-mirror Version: 2.37 Hello, I am maintaining a repository that consists of Debian packages merged with a couple of our modifications using reprepro. I have two codenames in the repo, beta3 and beta4, but actually the system behind this is Debian Stable, so I expect that the

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-22 Thread Christian PERRIER
Quoting Tokarski Boleslaw (boleslaw.tokar...@tieto.com): Thus long story short: I believe that choose-mirror should first check if the user has not specified the mirror/suite with preseed before setting his own default value. Good suggestion. Could you, by any chance, provide us with a

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-22 Thread Tokarski Boleslaw
Hello, I am no developer and I admit I do not know the very internals of the d-i system, but you can have a look at the attached patch. This checks if mirror/suite was already set and does not set the value in such a case in choose-mirror.postinst. I have also modified the choose-mirror.c

Bug#635053: Choose-mirror overrides mirror/suite and mirror/codename

2011-07-22 Thread Otavio Salvador
On Fri, Jul 22, 2011 at 12:58, Tokarski Boleslaw boleslaw.tokar...@tieto.com wrote: ... I will try to test this in my environment. Please test it and if it works ping us to review and commit it; regarding debootstrap you can preseed it using: d-i base-installer/debootstrap_script string squeeze