Bug 837837 - RFE: Reason palp binaries are installed as palp-${upstream_name} ?
Summary: RFE: Reason palp binaries are installed as palp-${upstream_name} ?
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: palp
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Conrad Meyer
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-07-05 14:26 UTC by Paulo Andrade
Modified: 2012-07-06 17:56 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-07-06 17:56:02 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Paulo Andrade 2012-07-05 14:26:03 UTC
To handle this, I will need to make some extra patches to my WIP sagemath package, but would like to know if there is really a requirement to name those this way...

A very good share of current "sage -testall" failures are caused by a message like:

Exception raised:
    [...]
    RuntimeError: Error executing "poly.x -fv" for a polytope sequence!
    Output:
    /bin/sh: poly.x: command not found

as the binary is actually installed as:

$ rpm -ql palp | grep poly.x
/usr/bin/palp-poly.x

I suggest a new package not renaming binaries, and also installing the arxiv.org documentation pdf for palp; not sure how to make automated tests on it tough, due to the way the cgi handles the requests for the file, see the new spec...

Suggested new package:
Spec URL: http://fedorapeople.org/~pcpa/palp.spec
SRPM URL: http://fedorapeople.org/~pcpa/palp-1.1-7.fc18.src.rpm

Comment 1 Conrad Meyer 2012-07-06 01:59:51 UTC
The rationale is here:

https://fedoraproject.org/wiki/Packaging:Conflicts#Potential_Conflicting_Files

There are only 4 binaries:
$ repoquery -ql palp | grep bin
/usr/bin/palp-class.x
/usr/bin/palp-cws.x
/usr/bin/palp-nef.x
/usr/bin/palp-poly.x

The names seem very short / generic, hence the palp- prefix. We could easily update the package to create a libexec directory with symlinks to the binaries, e.g.:

/usr/libexec/palp/class.x -> /usr/bin/palp-class.x
...

This would allow a user (or dependent package) to set PATH="$PATH:/usr/libexec/palp" and use the short binary names.

Do you think the names are unique enough?

Comment 2 Paulo Andrade 2012-07-06 14:37:36 UTC
I think they are unique enough, and would prefer to have less sagemath patches (already have way too much to adapt from sage release to sage release).
But could also add /usr/libexec/palp/ first to $PATH in the sage script.

Also, only people installing sagemath should also install palp anyway, but I still think there should not exist any other package with name conflicts.

Comment 3 Paulo Andrade 2012-07-06 14:42:18 UTC
I just asked for commit access to palp so I can do a more complete help on the issue :-)

Comment 4 Paulo Andrade 2012-07-06 17:56:02 UTC
New rawhide palp package corrects the problem.


Note You need to log in before you can comment on or make changes to this bug.