#985898 /usr/bin/r: /usr/bin/r is not stripped

Package:
r-cran-littler
Source:
littler
Description:
GNU R scripting and command-line front-end
Submitter:
Rogério Brito
Date:
2021-03-26 00:36:03 UTC
Severity:
wishlist
#985898#5
Date:
2021-03-25 16:12:42 UTC
From:
To:
Dear Dirk,

I was looking into some of my executables and discovered that /usr/bin/r is
not stripped and it contains debugging info on my system:

,----[ file /usr/bin/r ]
| /usr/bin/r: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=36aeb221c38bd7fdf3140a4c6481ff739573e634, for GNU/Linux 3.2.0, with debug_info, not stripped
`----

Is it desired? I would have expected it to work (as usual) with a -dbgsym
package being created if so needed (with all the justifications there).


Thanks for caring about R in Debian,

Rogério Brito.
#985898#10
Date:
2021-03-25 16:34:17 UTC
From:
To:
Rock, meet hard place.

The last change I made for Debian was just that: https://bugs.debian.org/968531

So I may have to close your bug report instead.

Dirk
#985898#15
Date:
2021-03-25 16:31:11 UTC
From:
To:
Howdy,

On 25 March 2021 at 13:12, Rogério Brito wrote:
| I was looking into some of my executables and discovered that /usr/bin/r is
| not stripped and it contains debugging info on my system:
|
| ,----[ file /usr/bin/r ]
| | /usr/bin/r: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, BuildID[sha1]=36aeb221c38bd7fdf3140a4c6481ff739573e634, for GNU/Linux 3.2.0, with debug_info, not stripped
| `----
|
| Is it desired? I would have expected it to work (as usual) with a -dbgsym
| package being created if so needed (with all the justifications there).

No that is likely an oversight as I need to "manually" copy the binary from
the R package location to /usr/bin.

common-binary-post-install-arch::
        dh_installdirs usr/bin usr/share/man/man1
        install -v -m 0644 $(debRlib)/littler/bin/r        $(CURDIR)/debian/$(package)/usr/bin/
        install -v -m 0644 $(debRlib)/littler/man-page/r.1 $(CURDIR)/debian/$(package)/usr/share/man/man1/

So yes -- that install call is lacking a '-s' flag.  Will add it now.

Thanks!

Dirk
#985898#20
Date:
2021-03-25 23:29:59 UTC
From:
To:
Dear Dirk,

I just saw your other message. I believe that, in the case of Debian, install, with or without -s may do the wrong thing.

Just copy the files where they belong to and debhelper will do the rest (it does for my packages).

I don't know how to integrate that cleanly with your upstream build system, though.

Thanks once again for packaging R,

Rogério Brito.
#985898#25
Date:
2021-03-26 00:32:38 UTC
From:
To:
On 25 March 2021 at 20:29, Rogério Theodoro de Brito wrote:
| On March 25, 2021 1:31:11 PM GMT-03:00, Dirk Eddelbuettel <edd@debian.org> wrote:
| >
| >Howdy,
| >
| >On 25 March 2021 at 13:12, Rogério Brito wrote:
| >| I was looking into some of my executables and discovered that
| >/usr/bin/r is
| >| not stripped and it contains debugging info on my system:
| >|
| >| ,----[ file /usr/bin/r ]
| >| | /usr/bin/r: ELF 64-bit LSB pie executable, x86-64, version 1
| >(SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2,
| >BuildID[sha1]=36aeb221c38bd7fdf3140a4c6481ff739573e634, for GNU/Linux
| >3.2.0, with debug_info, not stripped
| >| `----
| >|
| >| Is it desired? I would have expected it to work (as usual) with a
| >-dbgsym
| >| package being created if so needed (with all the justifications
| >there).
| >
| >No that is likely an oversight as I need to "manually" copy the binary
| >from
| >the R package location to /usr/bin.
| >
| >common-binary-post-install-arch::
| >        dh_installdirs usr/bin usr/share/man/man1
| >install -v -m 0644 $(debRlib)/littler/bin/r
| >$(CURDIR)/debian/$(package)/usr/bin/
| >install -v -m 0644 $(debRlib)/littler/man-page/r.1
| >$(CURDIR)/debian/$(package)/usr/share/man/man1/
| >
| >So yes -- that install call is lacking a '-s' flag.  Will add it now.
| >
| >Thanks!
| >
| >Dirk
|
| Dear Dirk,
|
| I just saw your other message. I believe that, in the case of Debian, install, with or without -s may do the wrong thing.
|
| Just copy the files where they belong to and debhelper will do the rest (it does for my packages).
|
| I don't know how to integrate that cleanly with your upstream build system, though.

I am not entirely sure I understand what you are suggesting, but the sources
are on salsa so feel free to experiment. If you find a better way, great. If
you do not, things stay the way they are. I will not have time to dig into
this for you anytime soon -- sorry.

Best, Dirk