Remove dangling symlink from JDK plugin #761

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
6 participants

gnuoy commented Aug 26, 2016

Using the JDK plugin results in a dangling symlink which in turn
causes lint-snap-v2_external_symlinks to fail:

package contains external symlinks:
usr/lib/jvm/java-8-openjdk-amd64/jre/lib/security/cacerts
lint-snap-v2_external_symlinks

This pull request removes the symlink from the snap
LP Bug #1617296

Remove dangling symlink from JDK plugin
Using the JDK plugin results in a dangling symlink which in turn
causes lint-snap-v2_external_symlinks to fail:

package contains external symlinks:
usr/lib/jvm/java-8-openjdk-amd64/jre/lib/security/cacerts
lint-snap-v2_external_symlinks

This pull request removes the symlink from the snap
LP Bug #1617296

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Collaborator

sergiusens commented Aug 26, 2016

Aren't the ca certs picked up from there though?

Collaborator

sergiusens commented Aug 26, 2016

Thanks for your contribution and bug report btw!

Collaborator

sergiusens commented Sep 20, 2016

Closing due to lack of activity, reopen please once you have time to look at this again.

@sergiusens sergiusens closed this Sep 20, 2016

davassi commented Nov 15, 2016

Just bumped across this issue. Could this pull request be merged?

Collaborator

sergiusens commented Nov 15, 2016

Once the question asked is answered, yes.

gnuoy commented Nov 15, 2016

Yes, the caerts are pulled in from there. I don't know what the correct way forward is for this issue.

@sergiusens sergiusens reopened this Nov 17, 2016

Member

elopio commented Nov 18, 2016

I don't see it dangling. I tried the java-hello-world demo that we have in snapcraft, and it links to the cert in etc. Can you please share the project where you are seeing this issue? ping @davassi @gnuoy
Thanks!

Member

kyrofa commented Dec 23, 2016

Closing again due to inactivity. Please reopen when you're ready to resume the conversation.

@kyrofa kyrofa closed this Dec 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment