+++ Johannes Schauer [2015-12-24 01:07 +0100]: > Hi, You've been a busy bee tonight! > > I'd be happy for sbuild-createchroot to switch to tar by default; > > please feel free to commit and upload such a change. > > unfortunately the current synopsis of sbuild-createchroot is: > > sbuild-createchroot [...] SUITE TARGET-DIRECTORY DEBIAN-MIRROR-URI > > If tarball creation were to become the default, then the TARGET-DIRECTORY > positional argument would become a tarball path instead. Would this not break > backwards compatibility with users/scripts assuming that this argument is a > directory? It is even assumed to be a directory when adding the > --make-sbuild-tarball option. > > So how about doing this instead: > > - promote the --make-sbuild-tarball option more in the EXAMPLES section of > the > sbuild-createchroot man page as it is already done in the sbuild wiki page. > > - when creating a directory based schroot, add union-type=overlay option by > default > > What do you think?
I think that would certianly be an improvement and agree that it would avoid incompatible/confusing changes. I'm not sure how much of a problem changing the meaning of that param from dir to tarball-basename actually is in practice. Perhaps it would still be worth it, to get a 'better' default. Doing it at t ime when lots of sbuild things are changing anyway might make sense... But I'm OK with taking the compatible route for now as that's easier and won't break anything. Wookey -- Principal hats: Linaro, Debian, Wookware, ARM http://wookware.org/
signature.asc
Description: Digital signature
