Bug#742672: allow debootstrap to take a bare codename as the script parameter
Control: tags -1 = patch On Sun, 27 May 2018, Hideki Yamane wrote: > control: tags -1 +pending Please don't mark the bug as pending when you submit a merge request. It means that the bug has a patch but not that the fix has been committed to git. BTW, I enabled a webhook that will do this automatically when commits are pushed to master and when they contain a "Closes: #xxx" in the commit message. > I've created Merge Request as > https://salsa.debian.org/installer-team/debootstrap/merge_requests/2 > > Please someone review it. Done. It's not the same as suggested in the bug. Cheers, -- Raphaël Hertzog ◈ Debian Developer Support Debian LTS: https://www.freexian.com/services/debian-lts.html Learn to master Debian: https://debian-handbook.info/get/
Processed: Re: Bug#742672: allow debootstrap to take a bare codename as the script parameter
Processing control commands: > tags -1 = patch Bug #742672 [debootstrap] allow debootstrap to take a bare codename as the script parameter Removed tag(s) pending. -- 742672: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=742672 Debian Bug Tracking System Contact ow...@bugs.debian.org with problems
Bug#742672: allow debootstrap to take a bare codename as the script parameter
control: tags -1 +pending Hi, I've created Merge Request as https://salsa.debian.org/installer-team/debootstrap/merge_requests/2 Please someone review it. -- Regards, Hideki Yamane henrich @ debian.org/iijmio-mail.jp
Bug#742672: allow debootstrap to take a bare codename as the script parameter
On Wed, 26 Mar 2014 06:32:06 + Tianon Gravi admwig...@gmail.com wrote: I've attached a patch that allows $4 (script) to be either an absolute path or a path/file within /usr/share/debootstrap/scripts, so that values like sid or wheezy can be supplied as the script argument and DWIM. ... Of course, as I noted on #742671, I'm very happy to adapt this to better match expected contribution style or scripting style. :) Friendly ping - anything I can do to help here? :) As I noted again on #742671, I'm still happy to make changes as necessary. :) ♥, - Tianon -- To UNSUBSCRIBE, email to debian-boot-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org Archive: https://lists.debian.org/20140905194001@gmail.com
Bug#742672: allow debootstrap to take a bare codename as the script parameter
Package: debootstrap Version: 1.0.59 Very closely related to #742671 (in which supplying sid for base-installer/debootstrap_script causes an error), I realized it might be ideal if the fix came in both directions. I've attached a patch that allows $4 (script) to be either an absolute path or a path/file within /usr/share/debootstrap/scripts, so that values like sid or wheezy can be supplied as the script argument and DWIM. Here's some proposed d/changelog text: [ Tianon Gravi ] * Allow script argument to be specified as either an absolute path as before, or as a raw codename such as sid, assumed to be relative to /usr/share/debootstrap/scripts. Of course, as I noted on #742671, I'm very happy to adapt this to better match expected contribution style or scripting style. :) ♥, - Tianon diff --git a/debootstrap b/debootstrap index fef1ab5..125e8bc 100755 --- a/debootstrap +++ b/debootstrap @@ -391,7 +391,11 @@ else SUPPORTED_VARIANTS=$VARIANT fi if [ $4 != ]; then - SCRIPT=$4 + if [ -e $DEBOOTSTRAP_DIR/scripts/$4 ]; then + SCRIPT=$DEBOOTSTRAP_DIR/scripts/$4 + else + SCRIPT=$4 + fi fi fi
