Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bugzilla DB migration completed. Please report issues to Infra team via email via infra@gentoo.org or IRC
Bug 685776 - net-misc/unison-2.51 USE=-ocamlopt strips bytecode from binaries
Summary: net-misc/unison-2.51 USE=-ocamlopt strips bytecode from binaries
Status: UNCONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2019-05-13 03:49 UTC by spock128
Modified: 2020-03-26 00:09 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
patch to disable stripping when not building native code (no-strip.patch,600 bytes, patch)
2019-05-13 03:49 UTC, spock128
Details | Diff
patch to disable stripping when not building native code (no-strip.patch,599 bytes, patch)
2019-05-13 04:02 UTC, spock128
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description spock128 2019-05-13 03:49:47 UTC
Created attachment 576256 [details, diff]
patch to disable stripping when not building native code

net-misc/unison with the -ocamlopt use flag seems to enable the -custom ocamlc flag in its build system.

According to the ocamlc man page, executables generated with -custom should not be stripped. Otherwise the bytecode will be removed and attempting to run unison will result in "No bytecode file specified." being displayed.

As a potential solution, I added this line to unison-2.51.2.ebuild

RESTRICT="!ocamlopt? ( strip )"

And also patched the unison Makefile to disable stripping (attachment).

The resulting (albeit unstripped) binary seems to work properly.
Comment 1 spock128 2019-05-13 04:02:12 UTC
Created attachment 576258 [details, diff]
patch to disable stripping when not building native code

ack flubbed the original patch