Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

patchelf lives in pkgsrc #25730

Merged
merged 1 commit into from Jul 27, 2015
Merged

patchelf lives in pkgsrc #25730

merged 1 commit into from Jul 27, 2015

Conversation

sjorge
Copy link
Contributor

@sjorge sjorge commented Jul 26, 2015

alteast on base-64-lts 14.4.0 release, no need download the binary from elsewhere.

alteast on base-64-lts 14.4.0 release
@jfindlay jfindlay added Documentation Relates to Salt documentation Packaging Related to packaging of Salt, not Salt's support for package management. bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch Minor Change Tests-Passed labels Jul 27, 2015
@jfindlay
Copy link
Contributor

@sjorge, thanks for the fix. In the future you are welcome to submit fixes directly to the oldest supported release branch, see https://docs.saltstack.com/en/latest/topics/development/contributing.html#which-salt-branch.

@nshalman

@sjorge
Copy link
Contributor Author

sjorge commented Jul 27, 2015

Should probably go in 2014.7 too then, not much use in 2015.x branches currently cut as both are broken on SmartOS.

@nshalman
Copy link
Contributor

+1 from me. I'm pretty sure patchelf wasn't in the pkgsrc builds I was using when I first did this work, but it is now so this is a definite improvement.

I also agree that it should ideally land in 2014.7 and get merged up to the other branches.

jfindlay added a commit that referenced this pull request Jul 27, 2015
@jfindlay jfindlay merged commit bd4540e into saltstack:develop Jul 27, 2015
@jfindlay
Copy link
Contributor

Thanks, @sjorge, @nshalman. We need to get SmartOS supported again on recent releases. Do either of you have specific enough information to file one or more issues on this? Have issues been filed for broken SmartOS support? @BretFisher, @fractaluniverse, and @johngrasty have recently been active on SmartOS issues with salt-bootstrap and may have additional input.

@nshalman
Copy link
Contributor

@jfindlay I haven't had a chance to look into the Esky issue, but to summarize an email thread with @dmurphy18:

@nshalman:

Something in 2015.5 doesn't seem to like being packaged in an Esky and I haven't had a chance to debug it...

@dmurphy18:

There is a problem with the use of decorators in the lazy loader where inspect.getmodule is not returning a module.
Found this with the 2015.5 branch, but have not had time to research a solution for the issue yet.
./salt/utils/decorators/__init__.py line 62
Found issues researching on Google about use of @decorator syntax and not using index 1 but 2 when inspecting the stack.
I have tried replacing the __init__.py file without the benefit of esky and the problem still occurred.
However esky could be affecting other files. BTW: python used was python 2.7.10.

My understanding is that @johngrasty was able to use salt-bootstrap to do a git based install of the 2015.5 branch.
I'm pretty sure that Joyent's pkgsrc builds package some version, but doesn't track as aggressively as I was tracking with my Esky builds.

@jfindlay
Copy link
Contributor

@nshalman, would you mind opening an issue with that info so we can track it? Thanks.

@sjorge sjorge deleted the patch-1 branch July 27, 2015 17:05
@sjorge
Copy link
Contributor Author

sjorge commented Jul 27, 2015

The find_library stuff in salt/utils/rsax931.py in 2015.8 also is broken on SmartOS esky package (maybe more as it tries to look for the system library, which is probably not available)

@nshalman
Copy link
Contributor

I've opened #25756.

@jfindlay
Copy link
Contributor

Thanks @nshalman.

@sjorge
Copy link
Contributor Author

sjorge commented Jul 27, 2015

Also made one for the find_library stuff in 2015.8 #25757

@rallytime rallytime added ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch. and removed bugfix-bckport will be be back-ported to an older release branch by creating a PR against that branch labels Jul 28, 2015
rallytime pushed a commit that referenced this pull request Jul 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Relates to Salt documentation Packaging Related to packaging of Salt, not Salt's support for package management. Tests-Passed ZZZ[Done]-back-ported-bf RETIRED The pull request has been back-ported to an older branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants