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

Switch path written to installservice.bat (part of Cassandra implementation) from Linux format to Windows format #271

Closed
sirredbeard opened this issue Dec 20, 2018 · 3 comments
Assignees
Labels
Bug This is a problem. Patched This has been fixed.
Milestone

Comments

@sirredbeard
Copy link

The path to autorun.bat (that auto-runs Cassandra) in installservice.bat (what installs autorun.bat as a Windows service) needs to be in Windows format, not Linux format, because it is running in cmd.

At https://github.com/WhitewaterFoundry/WLinux/blob/master/linux_files/setup#L1013

@sirredbeard sirredbeard added the Bug This is a problem. label Dec 20, 2018
@sirredbeard sirredbeard modified the milestones: 1.1.26, 1.1.28 Dec 20, 2018
@ghost
Copy link

ghost commented Dec 21, 2018

@patrick330602 do you have any plans to add WSLU support for Linux-->Windows path conversion? Otherwise, we can look into another fix.

@sirredbeard sirredbeard modified the milestones: 1.1.28, 1.1.29 Dec 27, 2018
@WSLUser
Copy link

WSLUser commented Jan 2, 2019

Any way to make them use Powershell instead (change .bat to .ps1)? Formatting wouldn't be an issue then since it's cross platform.

@sirredbeard
Copy link
Author

I would be fine with .ps1 files.

@sirredbeard sirredbeard modified the milestones: 1.1.29, 1.2 Jan 2, 2019
@sirredbeard sirredbeard added the To Implement This is something we should implement. label Mar 9, 2019
@sirredbeard sirredbeard modified the milestones: 1.2, 1.2.1 Mar 9, 2019
@crramirez crramirez self-assigned this Jun 19, 2019
@sirredbeard sirredbeard added Patched This has been fixed. and removed To Implement This is something we should implement. labels Aug 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a problem. Patched This has been fixed.
Projects
None yet
Development

No branches or pull requests

3 participants