Page MenuHomeSolus

Initial inclusion of patchelf
Needs ReviewPublic

Authored by tomocafe on Sun, Feb 2, 5:37 PM.

Details

Reviewers
Girtablulu
Group Reviewers
Triage Team
Maniphest Tasks
T8660: patchelf
Summary

Initial inclusion of patchelf. Resolves T8660.

Test Plan

Change the rpath of a binary

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

tomocafe created this revision.Sun, Feb 2, 5:37 PM
tomocafe requested review of this revision.Sun, Feb 2, 5:37 PM
tomocafe updated this revision to Diff 19537.Sun, Feb 2, 5:42 PM

Fix commit messages

Girtablulu requested changes to this revision.Sun, Feb 2, 5:42 PM
Girtablulu added a subscriber: Girtablulu.
Girtablulu added inline comments.
package.yml
9

should be below the source

13

shouldn't be on a new line

18

comes to the end

pspec_x86_64.xml
27

This can be removed

This revision now requires changes to proceed.Sun, Feb 2, 5:42 PM

and please a proper test, check while building is not enough

Jacalz added a subscriber: Jacalz.Sun, Feb 2, 5:53 PM
Jacalz added inline comments.
package.yml
9

Please always use https:// over http:// when it is supported.

tomocafe updated this revision to Diff 19539.Sun, Feb 2, 6:19 PM
tomocafe marked 4 inline comments as done.

package.yml fixes

tomocafe updated this revision to Diff 19540.Sun, Feb 2, 6:47 PM
tomocafe retitled this revision from Initial inclusion of patchelf. Resolves T8660. to Initial inclusion of patchelf.
tomocafe edited the summary of this revision. (Show Details)
tomocafe edited the test plan for this revision. (Show Details)
tomocafe added a task: T8660: patchelf.

Properly remove unneeded docs

tomocafe marked an inline comment as done.Sun, Feb 16, 11:29 PM

This one is ready for review, all feedback thus far has been addressed. Thanks!

package.yml
9

Done, also asked upstream to fix it on their docs. Copy pasted from them 😜

https://github.com/NixOS/patchelf/issues/191