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

KerbalAlarmClock install location #62

Closed
Ippo343 opened this issue Nov 18, 2014 · 5 comments
Closed

KerbalAlarmClock install location #62

Ippo343 opened this issue Nov 18, 2014 · 5 comments

Comments

@Ippo343
Copy link
Contributor

Ippo343 commented Nov 18, 2014

It has been pointed out on the forum that KAC is installed in the wrong folder: it should be in GameData/TriggerAU/KerbalAlarmClock, while presently it is in GameData/KerbalAlarmClock.

However the KerbalStuff release is just a copy of the zip posted to github, and thus has the same format:

KerbalAlarmClock.x.x.x.x/GameData/KerbalAlarmClock

Unfortunately to solve this we are going to need either the find keyword (KSP-CKAN/CKAN#376), or a way to install to a subdirectory of GameData (KSP-CKAN/CKAN#315).

@pjf
Copy link
Member

pjf commented Nov 19, 2014

Find keyword is absolutely what we want here. That's also going to give us joy for source releases KSP-CKAN/CKAN#259.

Find is already implemented internally by the default stanza code. We just need to make that a public method, test it, and bump the spec appropriately.

@btbonval
Copy link
Contributor

Ah, yeah I did that one and found myself limited in reproducing the hierarchy. Glad to know there's a fix.

@pjf
Copy link
Member

pjf commented Dec 11, 2014

With the new v1.2 pre-release, installing to a directory under GameData should now be operational.

@hakan42 hakan42 self-assigned this Dec 15, 2014
@hakan42
Copy link
Member

hakan42 commented Dec 15, 2014

Installing into a subdirectory of GameData works with my amended netkan, but the issue with the toplevel directory "KSPAlarmClock_x.y.z.q" still remains.

@Ippo343
Copy link
Contributor Author

Ippo343 commented Dec 24, 2014

Aaaaand now it's solved, thanks to the awesome work of @pjf :D

@Ippo343 Ippo343 closed this as completed Dec 24, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants