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

maven install should install the binaries in a usable way #51

Closed
displague opened this issue Apr 25, 2015 · 5 comments
Closed

maven install should install the binaries in a usable way #51

displague opened this issue Apr 25, 2015 · 5 comments

Comments

@displague
Copy link
Contributor

Following mvn clean install I currently symlink /src/CFLint/target/appassembler/bin/cflint to somewhere in my path (/.local/bin). I also have to make the binary executable chmod a+x ~/src/CFLint/target/appassembler/bin/cflint. These steps (setting permissions, and installing the appassembler artifacts) should be part of the install.

@ryaneberly
Copy link
Contributor

It looks like appassembler-maven-plugin resolved this.
http://jira.codehaus.org/browse/MAPPASM-54

Try using version 1.10 of the appassembler-maven-plugin in the cflint/pom.xml

@displague
Copy link
Contributor Author

The execute bit is set with 1.10 but that doesn't address how one gets the appassembler artifacts into usable locations.

@displague
Copy link
Contributor Author

Does it have something to do with:
http://mojo.codehaus.org/appassembler/appassembler-maven-plugin/create-repository-mojo.html#assembleDirectory
is there a way to mvn clean install --assembleDirectory=~/.local (or /usr/local/ or /opt/ as others may choose) ?

@ryaneberly
Copy link
Contributor

you can do that:
mvn clean install -DassembleDirectory=~/.local

it will create files in bin/ lib/ and etc/ folders.

I'm not necessarily recommending this. We may need to leave that last step up the user.

ryaneberly added a commit that referenced this issue Apr 27, 2015
@displague
Copy link
Contributor Author

Thanks! mvn clean install -D assembleDirectory=~/.local worked (had to remove the dash after -D).

I think it's worth noting in the build instructions (perhaps with a more common directory like /usr/local/).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants