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

Fix issue #43 #51

Closed
wants to merge 3 commits into from
Closed

Fix issue #43 #51

wants to merge 3 commits into from

Conversation

laguiz
Copy link
Contributor

@laguiz laguiz commented Oct 15, 2013

Windows MSI + Play 2.2.0 : CNDL0027 : The Shortcut/@name attribute's value, '\application.conf', is not a valid long name because it contains illegal characters

Fix issue #43 : #43
…wrong DirectoryRef ID

Make sure we only have one '/' for each separator and no '/' at the end.
@laguiz
Copy link
Contributor Author

laguiz commented Oct 15, 2013

PR submitted

@jsuereth
Copy link
Member

Closing in favor of #50 which includes the three commits from this pull request. Do you mind checking the additions I added? Besides merging into master, there's one commit of activity there.

@laguiz
Copy link
Contributor Author

laguiz commented Oct 15, 2013

Sorry I did not check before. I'm going to look #50

@jsuereth jsuereth closed this Oct 16, 2013
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

Successfully merging this pull request may close these issues.

2 participants