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

Avoid having two effekt binaries (deprecate effekt.sh) #166

Merged
merged 2 commits into from
Oct 26, 2022

Conversation

b-studios
Copy link
Collaborator

Previously we had to have two commands:

  • effekt which worked on MacOs
  • effekt.sh which worked on Ubuntu systems

in #16 @waterlens proposes to use -S (which is short for --split-string).

I am not sure anymore, why I didn't do that in the first place. Maybe -S is not available on some implementations?

Could someone with Ubuntu please try whether this works? Simply run

sbt assembleBinary

and then there should be bin/effekt which you probably need to chmod +x and then execute. Can you also try executing it with some arguments (like --help)?

@b-studios b-studios changed the title Append -S as proposed by @waterlens in #17 Avoid having two effekt binaries (drop effekt.sh) Oct 24, 2022
@b-studios b-studios changed the title Avoid having two effekt binaries (drop effekt.sh) Avoid having two effekt binaries (deprecate effekt.sh) Oct 24, 2022
Copy link
Collaborator

@phischu phischu left a comment

Choose a reason for hiding this comment

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

Just works on Ubuntu 20.04

@b-studios b-studios merged commit faf8659 into master Oct 26, 2022
@b-studios b-studios deleted the fix/effekt-cmd-on-linux branch October 26, 2022 09:55
@b-studios
Copy link
Collaborator Author

b-studios commented Nov 28, 2022

Looks like this has introduced a couple of regressions... We just got reports that it does break on Ubuntu 22.04. Also @JonathanStarup reported breakage on Windows.

@b-studios
Copy link
Collaborator Author

Maybe someone is curious to look into https://github.com/sbt/sbt-native-packager ?

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.

None yet

2 participants