Skip to content
This repository has been archived by the owner on Jul 5, 2019. It is now read-only.

#11 First work done #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

#11 First work done #12

wants to merge 2 commits into from

Conversation

muuki88
Copy link

@muuki88 muuki88 commented Sep 21, 2013

  • Start script only works when you start it on the correct path. Play
    starter script doesn't evaluate the relative symlink correctly
  • init.d script doesn't work correctly even when the path is fixed (hard
    coded). No idea here.

- Start script only works when you start it on the correct path. Play
starter script doesn't evaluate the relative symlink correctly

- init.d script doesn't work correctly even when the path is fixed (hard
coded). No idea here.
@muuki88
Copy link
Author

muuki88 commented Sep 21, 2013

The sbt-native-packager script is here

It is used in the JavaAppBashScript.scala

@kryptt
Copy link
Owner

kryptt commented Sep 21, 2013

Cool!

packageSummary in Debian <<= packageSummary,
name in Debian <<= normalizedName,
// To be conform with the sbt-native-packager
fullName <<= (name, version) apply (_ + "-" + _),
Copy link
Owner

Choose a reason for hiding this comment

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

This probably doesn't do what you would expec.
Do fullname in Debian as well as use the same for name and version.

@muuki88
Copy link
Author

muuki88 commented Sep 21, 2013

The problem with the bash script seems to be that realpath() cannot handle the relative symlink correctly.

/usr/bin/<app> -> ../share/<app>/bin/<app>

when I run ./<app> from /usr/share/<app>/bin everything works. On every other path it fails, as it cannot find the /usr/share/<app>/lib aka the lib_dir. I think I will open a bug on the sbt-native-packager issue tracker.

@muuki88
Copy link
Author

muuki88 commented Sep 21, 2013

Ticket sbt/sbt-native-packager#39

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

Successfully merging this pull request may close these issues.

2 participants