search

ARM-software / gator

Sources for Arm Streamline's gator daemon, part of Arm Mobile Studio suite of performance analysis tools
https://www.arm.com/products/development-tools/graphics/arm-mobile-studio
133 stars 70 forks source link

gator: don't strip debug #26

Closed clementperon closed 3 years ago

clementperon commented 3 years ago

Hi,

I plan to push a recipe for gator daemon to yocto meta-arm layer, and yocto already split debug symbols out of target bin.

I would like to know if there is any reason to strip debug symbols in the final executable?

Clement

rossburton commented 3 years ago

As a distro maintainer, I endorse this patch: don't strip on install, we'll do that ourselves.

bengaineyarm commented 3 years ago

Hi Clement, thanks for the PR.

I would like to know if there is any reason to strip debug symbols in the final executable?

No especially strong reason.

Happy to accept this, but I'm going to wait till after 7.7 drops as it is imminent.

solidpixel commented 3 years ago

One note for anyone packaging this for use in a distro is that the version of gator is specific to a version of the Streamline host tool, so packaging this in a distro will either lock users to an old version of the host tool or require the them to ignore the distro packaged version and replace it with the latest version from the tool.

bengaineyarm commented 3 years ago

Yes, good point @solidpixel - if your distro package manager has the concept of parallel versions / slots allowing multiple versions of a thing to be installed at the same time, it might be worth using that feature and perhaps installing the binary as gator-x.x instead of gator. Alternatively if possible, provide multiple versions in the repo, but require user to downgrade/upgrade.