Debian Bug report logs - #331554
--variant compatibility option

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Junichi Uekawa <dancer@netfort.gr.jp>

To: submit@bugs.debian.org

Subject: --variant compatibility option

Date: Tue, 04 Oct 2005 08:50:40 +0900

Package: cdebootstrap
Version: 0.3.8

Hi,

Please add a --variant alias to --flavor option, so that 
cdebootstrap can be used command-line compatible.

It will be required when etch releases with 
pbuilder supporting cdebootstrap; and users upgrade from
sarge with configuration file using debootstrap.


regards,
	junichi

Message #10 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Bastian Blank <waldi@debian.org>

To: Junichi Uekawa <dancer@netfort.gr.jp>, 331554@bugs.debian.org

Subject: Re: Bug#331554: --variant compatibility option

Date: Tue, 4 Oct 2005 12:08:24 +0200

[Message part 1 (text/plain, inline)]

On Tue, Oct 04, 2005 at 08:50:40AM +0900, Junichi Uekawa wrote:
> Please add a --variant alias to --flavor option, so that 
> cdebootstrap can be used command-line compatible.

This options are not compatible. Even the arguments don't match.

Bastian

-- 
Dammit Jim, I'm an actor, not a doctor.

[signature.asc (application/pgp-signature, inline)]

Message #15 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Junichi Uekawa <dancer@netfort.gr.jp>

To: Bastian Blank <waldi@debian.org>

Cc: Junichi Uekawa <dancer@netfort.gr.jp>, 331554@bugs.debian.org

Subject: Re: Bug#331554: --variant compatibility option

Date: Tue, 04 Oct 2005 20:59:03 +0900

Hi,

> > Please add a --variant alias to --flavor option, so that 
> > cdebootstrap can be used command-line compatible.
> 
> This options are not compatible. Even the arguments don't match.

Either way would work:

1. making it a noop option

2. translating the options to the similar desired value.
 AFAIK, fakechroot is unsupported, and buildd option should be 
 easily translated.


regards,
	junichi

Message #20 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Bastian Blank <waldi@debian.org>

To: Junichi Uekawa <dancer@netfort.gr.jp>, 331554@bugs.debian.org

Subject: Re: Bug#331554: --variant compatibility option

Date: Wed, 5 Oct 2005 11:23:02 +0200

[Message part 1 (text/plain, inline)]

On Tue, Oct 04, 2005 at 08:50:40AM +0900, Junichi Uekawa wrote:
> It will be required when etch releases with 
> pbuilder supporting cdebootstrap; and users upgrade from
> sarge with configuration file using debootstrap.

I currently don't see, why this should be a problem with cdebootstrap.
The commandline interface is not compatible.

Bastian

-- 
The more complex the mind, the greater the need for the simplicity of play.
		-- Kirk, "Shore Leave", stardate 3025.8

[signature.asc (application/pgp-signature, inline)]

Message #25 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Junichi Uekawa <dancer@netfort.gr.jp>

To: Bastian Blank <waldi@debian.org>

Cc: Junichi Uekawa <dancer@netfort.gr.jp>, 331554@bugs.debian.org

Subject: Re: Bug#331554: --variant compatibility option

Date: Wed, 05 Oct 2005 21:35:42 +0900

Hi,

> > It will be required when etch releases with 
> > pbuilder supporting cdebootstrap; and users upgrade from
> > sarge with configuration file using debootstrap.
> 
> I currently don't see, why this should be a problem with cdebootstrap.
> The commandline interface is not compatible.

They are mostly compatible; and pbuilder uses 
debootstrap and cdebootstrap interchangably; 
and most notably, the switch of default has happened 
very recently.

regards,
	junichi

Message #30 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Junichi Uekawa <dancer@netfort.gr.jp>

To: Bastian Blank <waldi@debian.org>

Cc: Junichi Uekawa <dancer@netfort.gr.jp>, 331554@bugs.debian.org, Bug tracking control <control@bugs.debian.org>

Subject: Re: Bug#331554: --variant compatibility option

Date: Sun, 09 Oct 2005 21:02:19 +0900

tags 331554 + patch
thanks


patch follows,


regards,
	junichi

diff -ur cdebootstrap-0.3.9orig/src/frontend/standalone/cdebootstrap.man cdebootstrap-0.3.9-dif/src/frontend/standalone/cdebootstrap.man
--- cdebootstrap-0.3.9orig/src/frontend/standalone/cdebootstrap.man	2005-02-10 20:56:05.000000000 +0900
+++ cdebootstrap-0.3.9-dif/src/frontend/standalone/cdebootstrap.man	2005-10-09 20:58:00.000000000 +0900
@@ -25,6 +25,10 @@
 .B FLAVOURS
 below).
 .TP
+\fB\-\-variant\fR=\fIVARIANT\fR
+Compatibility option against debootstrap. supports
+\fBbuildd \fR only.
+.TP
 \fB\-q\fR, \fB\-\-quiet\fR
 be quiet
 .TP
diff -ur cdebootstrap-0.3.9orig/src/frontend/standalone/main.c cdebootstrap-0.3.9-dif/src/frontend/standalone/main.c
--- cdebootstrap-0.3.9orig/src/frontend/standalone/main.c	2005-01-06 06:48:37.000000000 +0900
+++ cdebootstrap-0.3.9-dif/src/frontend/standalone/main.c	2005-10-09 20:56:29.000000000 +0900
@@ -63,6 +63,7 @@
   GETOPT_DEBUG,
   GETOPT_DEVFS,
   GETOPT_VERSION,
+  GETOPT_VARIANT
 };
 
 static struct option const long_opts[] =
@@ -72,6 +73,7 @@
   {"debug", no_argument, 0, GETOPT_DEBUG},
   {"download-only", no_argument, 0, 'd'},
   {"flavour", required_argument, 0, 'f'},
+  {"variant", required_argument, 0, GETOPT_VARIANT},
   {"helperdir", required_argument, 0, 'H'},
   {"include", required_argument, 0, 'i'},
   {"quiet", no_argument, 0, 'q'},
@@ -146,6 +148,17 @@
   exit (status);
 }
 
+char *debootstrap_compat_match_flavor(const char *matchstr)
+{
+  /* compatibility with debootstrap */
+  if (!strcmp(matchstr,"buildd"))
+    return strdup("build");
+  else if (!strcmp(matchstr,"fakechroot"))
+    return NULL;
+  else
+    return NULL;
+}
+
 int frontend_main (int argc, char **argv, char **envp)
 {
   int c;
@@ -156,7 +169,7 @@
 
   program_name = argv[0];
 
-  while ((c = getopt_long (argc, argv, "a:c:df:hH:i:qv", long_opts, NULL)) != -1)
+  while ((c = getopt_long (argc, argv, "a:c:df:hH:i:qvV:", long_opts, NULL)) != -1)
   {
     switch (c)
     {
@@ -215,6 +228,13 @@
 ", stdout);
         exit (EXIT_SUCCESS);
         break;
+      case GETOPT_VARIANT:
+        if (!(flavour = debootstrap_compat_match_flavor(optarg)))
+	{
+	  log_text(DI_LOG_LEVEL_ERROR, "Invalid flavour");
+	  exit (EXIT_FAILURE);
+	}
+        break;
       default:
         usage (EXIT_FAILURE);
     }

Message #37 received at 331554@bugs.debian.org (full text, mbox, reply):

From: Junichi Uekawa <dancer@netfort.gr.jp>

To: Andreas Beckmann <debian@abeckmann.de>, 351536@bugs.debian.org, 331554@bugs.debian.org

Subject: Re: [Pbuilder-maint] Bug#351536: please document cdebootstrap --flavour=build in pbuilderrc

Date: Tue, 07 Feb 2006 19:58:38 +0900

Hi,

> pbuilderrc contains:
> ...
> # Set the debootstrap variant to 'buildd' type.
> # DEBOOTSTRAPOPTS[0]='--variant=buildd'
> # or work around bug in debootstrap 3.0.0 (314858)
> unset DEBOOTSTRAPOPTS
> ...
> - this should probably mean debootstrap 0.3.0 ?
> - now that debootstrap is fixed, this workaround could be disabled ?
> 
> But since the default DEBOOTSTRAP command is cdebootstrap, the proper
> option to create a buildd like chroot with cdebootstrap should be
> documented in the pbuilderrc, too. Perhaps this should be enabled by
> default:

I'm waiting for 331554 to be resolved.



regards,
	junichi
-- 
dancer@{debian.org,netfort.gr.jp}   Debian Project

Message #42 received at 331554-close@bugs.debian.org (full text, mbox, reply):

From: Bastian Blank <waldi@debian.org>

To: 331554-close@bugs.debian.org

Subject: Bug#331554: fixed in cdebootstrap 0.3.11

Date: Sun, 26 Feb 2006 05:32:06 -0800

Source: cdebootstrap
Source-Version: 0.3.11

We believe that the bug you reported is fixed in the latest version of
cdebootstrap, which is due to be installed in the Debian FTP archive:

cdebootstrap-udeb_0.3.11_i386.udeb
  to pool/main/c/cdebootstrap/cdebootstrap-udeb_0.3.11_i386.udeb
cdebootstrap_0.3.11.dsc
  to pool/main/c/cdebootstrap/cdebootstrap_0.3.11.dsc
cdebootstrap_0.3.11.tar.gz
  to pool/main/c/cdebootstrap/cdebootstrap_0.3.11.tar.gz
cdebootstrap_0.3.11_i386.deb
  to pool/main/c/cdebootstrap/cdebootstrap_0.3.11_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 331554@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Bastian Blank <waldi@debian.org> (supplier of updated cdebootstrap package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Sun, 26 Feb 2006 14:12:59 +0100
Source: cdebootstrap
Binary: cdebootstrap cdebootstrap-udeb
Architecture: source i386
Version: 0.3.11
Distribution: unstable
Urgency: low
Maintainer: Bastian Blank <waldi@debian.org>
Changed-By: Bastian Blank <waldi@debian.org>
Description: 
 cdebootstrap - Bootstrap a Debian system
 cdebootstrap-udeb - Bootstrap a Debian system (udeb)
Closes: 331554 332907
Changes: 
 cdebootstrap (0.3.11) unstable; urgency=low
 .
   * config:
     - Build dev for minimal flavour.
   * Implement --variant compatibility option. (closes: #331554)
   * Fix generic error message. (closes: #332907)
Files: 
 3d9bc339f8dc8844a4eac0507f66eb5a 630 admin optional cdebootstrap_0.3.11.dsc
 70146d0d7c8e9755dd47dda128277710 127861 admin optional cdebootstrap_0.3.11.tar.gz
 662c6e90b4228f1982023adbc074a378 24166 admin optional cdebootstrap_0.3.11_i386.deb
 ac0080780fe37c9d4f4d607a375cbf16 16304 debian-installer optional cdebootstrap-udeb_0.3.11_i386.udeb
Package-Type: udeb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iEYEARECAAYFAkQBqdEACgkQLkAIIn9ODhG1QQCgs/cw2sWlxrGu62LMNbUL77Fs
CxkAoLry2IldH78RbW+U3DT7KfFnFKCI
=1DGM
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.