Bug#968529: s3switch: probably shouldn't use install -s
> > I maintain the code at g...@salsa.debian.org:tormod-guest/s3switch.git > Can you please create the repo at > g...@salsa.debian.org:xorg-team/app/s3switch.git ? > Scratch that, the repo is already there now. But I cannot push to it. If anyone can pull from mine and push there it would be good. Tormod
Bug#968529: s3switch: probably shouldn't use install -s
tag 968529 pending thanks Hi Simon, Thanks for the report. I chose the latter of your suggestions (and added dh_install). I maintain the code at g...@salsa.debian.org:tormod-guest/s3switch.git Can you please create the repo at g...@salsa.debian.org:xorg-team/app/s3switch.git ? Best regards, Tormod On Mon, Aug 17, 2020 at 12:51 AM Simon McVittie wrote: > > d/rules in s3switch sets INSTALL_PROGRAM to 'install -s'. However, > invoking 'install -s' is usually only harmful for a Debian package, > not helpful: it discards debugging symbols that could otherwise > have gone into a -dbgsym package, and when cross-compiling it uses > the wrong strip(1) implementation (from the build rather than host > architecture). Unconditionally using INSTALL_PROGRAM = install would > likely be better. Or, you could just use > > s3switch usr/bin > > in debian/s3switch.install, and you wouldn't need the install step at all. > > smcv
Bug#968529: s3switch: probably shouldn't use install -s
Source: s3switch Version: 0.1-1 Severity: normal A bug report against fteqcc (#968524) prompted me to look for other instances of the same anti-pattern on codesearch.debian.net. d/rules in s3switch sets INSTALL_PROGRAM to 'install -s'. However, invoking 'install -s' is usually only harmful for a Debian package, not helpful: it discards debugging symbols that could otherwise have gone into a -dbgsym package, and when cross-compiling it uses the wrong strip(1) implementation (from the build rather than host architecture). Unconditionally using INSTALL_PROGRAM = install would likely be better. Or, you could just use s3switch usr/bin in debian/s3switch.install, and you wouldn't need the install step at all. smcv