Frans Pop [EMAIL PROTECTED] writes:
On Saturday 01 March 2008, Otavio Salvador wrote:
+# Check if a specific script should be used
+db_get base-installer/debootstrap_script
+DEBOOTSTRAP_SCRIPT=$RET || true
What's the point of the || true here?
IMO the patch should test that the script
On Sunday 02 March 2008, Otavio Salvador wrote:
Frans Pop [EMAIL PROTECTED] writes:
On Saturday 01 March 2008, Otavio Salvador wrote:
+# Check if a specific script should be used
+db_get base-installer/debootstrap_script
+DEBOOTSTRAP_SCRIPT=$RET || true
What's the point of the || true
Jérémy Bobbio [EMAIL PROTECTED] writes:
On Sat, Mar 01, 2008 at 06:17:25PM -0300, Otavio Salvador wrote:
It's possible to use a specific script by preseeding
base-installer/script. That's a very interesting feature for Custom
Debian Distributions and derivatives to avoid to maintain a fork of
Frans Pop [EMAIL PROTECTED] writes:
On Sunday 02 March 2008, Otavio Salvador wrote:
Frans Pop [EMAIL PROTECTED] writes:
On Saturday 01 March 2008, Otavio Salvador wrote:
+# Check if a specific script should be used
+db_get base-installer/debootstrap_script
+DEBOOTSTRAP_SCRIPT=$RET ||
It's possible to use a specific script by preseeding
base-installer/script. That's a very interesting feature for Custom
Debian Distributions and derivatives to avoid to maintain a fork of
debootstrap package.
---
.../base-installer/debian/bootstrap-base.postinst |7 +++
On Sunday 02 March 2008, Otavio Salvador wrote:
+if [ ! -e /usr/share/debootstrap/scripts/$DEBOOTSTRAP_SCRIPT ]; then
+ error '$DEBOOTSTRAP_SCRIPT' debootstrap script doesn't exist.
+fi
That should be:
+ error debootstrap script '$DEBOOTSTRAP_SCRIPT' doesn't exist
Reversed order
It's possible to use a specific script by preseeding
base-installer/script. That's a very interesting feature for Custom
Debian Distributions and derivatives to avoid to maintain a fork of
debootstrap package.
---
.../base-installer/debian/bootstrap-base.postinst |7 +++
It's possible to use a specific script by preseeding
base-installer/script. That's a very interesting feature for Custom
Debian Distributions and derivatives to avoid to maintain a fork of
debootstrap package.
---
.../base-installer/debian/bootstrap-base.postinst |4
On Saturday 01 March 2008, Otavio Salvador wrote:
+# Check if a specific script should be used
+db_get base-installer/debootstrap_script
+DEBOOTSTRAP_SCRIPT=$RET || true
What's the point of the || true here?
IMO the patch should test that the script exists and, if not, log the fact
and
On Sat, Mar 01, 2008 at 06:17:25PM -0300, Otavio Salvador wrote:
It's possible to use a specific script by preseeding
base-installer/script. That's a very interesting feature for Custom
Debian Distributions and derivatives to avoid to maintain a fork of
debootstrap package.
I personally have
10 matches
Mail list logo