On Fri, 24 Jan 2020, Anuradha Weeraman wrote: > Thanks for the feedback. > > Happy to receive a PR and support with uploading ksh93. Will replicate > accordingly to ksh2020 as well.
You’re welcome. Just to be clear: > > > Thanks. Are you going to package ksh93 as well, or should > > > I do that (I’m a DD and can do it easily)? > You can find a branch on Salsa for ksh93 with the alternatives support here: > https://salsa.debian.org/debian/ksh/tree/debian/ksh93 > I've done some co-existence testing on it. If you could review and help > sponsor the package that would be awesome. Should I review, possibly fix up, and then team-upload it? > > More questions, from looking at the debdiff: > > > > >> * Use alternative system for ksh, shcomp and rksh > > > > You’re setting these up as slaves of /bin/ksh, which however > > does not work if the user chooses /bin/ksh to be mksh but > > still wants to have shcomp for ksh93 or ksh2020 available. > > > > I’d separate at _least_ shcomp into its own alternative, > > if not rksh (which _can_ be configured separately from ksh, > > but does not need to; currently, mksh does not provide rksh, > > but in the future it might). > > > > +++ ksh-2020.0.0/debian/gitlab-ci.yml 2020-01-19 07:43:52.000000000 +0100 > > + SALSA_CI_PIUPARTS_ARGS: --ignore=/etc/shells > > > > This will not work in practice: piuparts as run by britney > > (Debian testing migration) doesn’t read this, and so it will > > never migrate. The cause is easy: > > > > +++ ksh-2020.0.0/debian/ksh.postinst 2020-01-19 07:43:52.000000000 +0100 > > +add-shell /bin/ksh > > +add-shell /bin/rksh > > > > Please *don’t* add /bin/ksh (and /bin/rksh) to /etc/shells; > > both ksh before your change and mksh don’t do that, and we > > decided some time back to let the user only add explicit > > choices, not things from the alternative system, as shell, > > so they keep their shell even when the alternative changes. > > > > Please also remove the comment referring #790118 there, as > > it was superceded by the above discussion. > > > > Doing this will fix piuparts at the same time ☻ > > > > shcomp2020.1 and 0001-List-only-supported-names-for-ksh.patch > > also need the new names. > > > > Do you want me to prepare a pull request with these changes, > > and, possibly, upload them (to save you effort)? Just to be clear: should I only prepare a pull request, or should I also team-upload the changes? bye, //mirabilos (mksh developer and packager; Debian Developer) -- tarent solutions GmbH Rochusstraße 2-4, D-53123 Bonn • http://www.tarent.de/ Tel: +49 228 54881-393 • Fax: +49 228 54881-235 HRB 5168 (AG Bonn) • USt-ID (VAT): DE122264941 Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg ********** Mit der tarent Academy bieten wir auch Trainings und Schulungen in den Bereichen Softwareentwicklung, Agiles Arbeiten und Zukunftstechnologien an. Besuchen Sie uns auf www.tarent.de/academy. Wir freuen uns auf Ihren Kontakt. **********
