search

cbeust / kobalt

A Kotlin-based build system for the JVM.
Apache License 2.0
433 stars 60 forks source link

make kobaltw executable by default #401

Closed pabl0rg closed 7 years ago

pabl0rg commented 7 years ago

When Kobalt upgrades, it unzips the new version of the script to ~/.kobalt/wrapper/dist/kobalt-<version>/bin

However, it is not executable (on mac at least). Meaning that if I want to use that script to init a new project, I first have to make it executable.

>ls -l ~/.kobalt/wrapper/dist/kobalt-1.0.53/bin/
total 16
-rw-r--r--  1 jliska  staff  197 Apr  6 12:47 kobaltw
-rw-r--r--  1 jliska  staff  121 Apr  6 12:47 kobaltw.bat
ethauvin commented 7 years ago

Yes, I've noticed that too. I'll look into it, should be fixable.

ethauvin commented 7 years ago

@pabl0rg Do you have the same issue with the latest release?

pabl0rg commented 7 years ago

@ethauvin Yes.

I guess this wouldn't be an issue if there is/was a better way to init Kobalt in a project . The homebrew version wasn't working for me last time I tried (a long time ago) and either way, if I init with the homebrew version, I know I'll get an old kobalt and have to update. So, I either install kobalt by running an existing project's kobaltw or dig around ~/.kobalt/wrapper/dist

ls -l ~/.kobalt/wrapper/dist/kobalt-1.0.59/bin/
total 16
-rw-r--r--  1 juanliska  staff  203 Apr 10 02:17 kobaltw
-rw-r--r--  1 juanliska  staff  125 Apr 10 02:17 kobaltw.bat
ethauvin commented 7 years ago

I don't get that at all on my Windows box:

$ ls -l ~/.kobalt/wrapper/dist/kobalt-1.0.59/bin/
total 2.0K
-rwxr-xr-x 1 erik 197609 203 Apr  9 11:29 kobaltw
-rw-r--r-- 1 erik 197609 125 Apr  9 11:29 kobaltw.bat

@cbeust will test on his Mac.

ethauvin commented 7 years ago

Actually, just saw the same on my Linux box. Will investigate some more.