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

Solaris: Fix IPS package must create symlinks to package commands #5485

Merged
merged 1 commit into from
Nov 1, 2016

Conversation

jaymalasinha
Copy link
Contributor

Description

Create symlinks.erb template resource for chef IPS package so it will install the symbolic links for the commands. These are the links added by post install scripts for other package types. But since IPS does not use post install scripts they need to be added here.

Resolves: #5458 after successful merge for this PR and chef/omnibus#728

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]

Check List

Create symlinks.erb template to exists in local project directory structure
to enable IPS package to create required symlinks for installed commands

Signed-off-by: Jaymala Sinha jsinha@chef.io

Create symlinks.erb template to exists in local project directory structure
to enable IPS package to create required symlinks for installed commands

Signed-off-by: Jaymala Sinha <jsinha@chef.io>
@jaymalasinha jaymalasinha force-pushed the jsinha/BEN-123-IPSFIX branch from 87c657f to 42766ae Compare October 24, 2016 17:29
@@ -0,0 +1,6 @@
link path=usr/bin/chef-solo target=<%= projectdir %>/bin/chef-solo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems odd this is usr/bin/chef-solo vs /usr/bin/chef-solo...is that intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. It creates it relative to /

@tas50 tas50 added the Bug label Oct 28, 2016
@thommay thommay merged commit cfb6e40 into master Nov 1, 2016
@lamont-granquist lamont-granquist changed the title [BEN-123] Fix IPS package must create symlinks to package commands Solaris: Fix IPS package must create symlinks to package commands Nov 3, 2016
@thommay thommay added Type: Bug Does not work as expected. and removed Bug labels Jan 25, 2017
@thommay thommay deleted the jsinha/BEN-123-IPSFIX branch January 31, 2017 10:31
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Solaris IPS packages are missing some symlinks found in the pre-IPS packages
6 participants