Bug#933299: isohybrid.pl should not be architecture dependent

2019-07-29 Thread Thomas Schmitt
Hi,

>  I'll also look into whether we need both the perl
> script and the binary of "isohybrid"

If it is for functionality, then not.

If it is about portability, then only if isohybrid.c stays restricted to
the architectures which could boot via isolinux.bin.

If it is about backward compatibility to old scripts, then the name with
the suffix .pl would need to be kept.
The help text option list of .pl is a subset of the one of .c. The number
of dashes with long options differs. But the use of getopt_long_only(3)
in isohybrid.c compensates for this difference:
  $ /usr/bin/isohybrid.pl --entry 1 isolinux.iso
  Usage: /usr/bin/isohybrid.pl [options] filename.iso
  ...
  $ /usr/bin/isohybrid.pl -entry 1 isolinux.iso
  $ /usr/bin/isohybrid --entry 1 isolinux.iso
  $ /usr/bin/isohybrid -entry 1 isolinux.iso
  $


Have a nice day :)

Thomas



Bug#933299: isohybrid.pl should not be architecture dependent

2019-07-28 Thread Lukas Schwaighofer
Hi Timothy,

thanks for reporting.  The whole file organization of syslinux (both
which file is in which package, as well as the location of files) needs
improvement.  This is something I intend to work on during the bullseye
cycle.

As part of that, I'll also look into whether we need both the perl
script and the binary of "isohybrid" and in which package they should
be placed.  It'll still be a while until I get to it though (most
likely towards the end of this year).


@Thomas: If I remember correctly, isohybrid.pl only needs perl-base
(which is an essential package and does not require a dependency).
I'll double check that too.

Regards
Lukas



Bug#933299: isohybrid.pl should not be architecture dependent

2019-07-28 Thread Thomas Schmitt
Hi,

shouldn't the package which offers isohybrid.pl depend on perl ?
Note that isohybrid.pl prepares an ISOLINUX ISO only for BIOS booting.
Not for EFI, as does isohybrid from the C source.

isohybrid from isohybrid.c would have to be in a package with
  Architecture: any
to produce binary executables for each CPU type.
"Architecture: all" produces only one binary package for all CPUs.
syslinux debian/control file has no "Architecture: any" package yet.


Have a nice day :)

Thomas



Bug#933299: isohybrid.pl should not be architecture dependent

2019-07-28 Thread Timothy Pearson
Package: syslinux-utils
Version: 3:6.04~git20190206.bf6db5b4+dfsg1-1

There are two versions of the hybrid ISO utility, a native C implementation and 
a Perl version.  Neither of these should be in the architecture-dependent 
(x86-only) syslinux-utils package, but especially the Perl version of the 
script should be moved to syslinux-common (which is architecture-independent).