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

DataPath DefaultValue is Windows-specific #2

Open
robpvn opened this Issue Jun 30, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@robpvn
Copy link
Contributor

robpvn commented Jun 30, 2012

See https://github.com/slluis/cydin/blob/master/Cydin/Models/Settings.cs#L29

This value leads, when run on a Linux system for the first time, to a Folder named "..\Files" being created in the Cydin folder, instead of "Files" in the parent folder as intended. This leads to the generation of add-in repositories screwing up. It's slightly annoying to modify the Configuration table in the database to change this.

It's a simple fix to hardcode it otherwise, but since it's a Datamember DefaultValue you can't simply use IO.Path.DirectorySeperatorChar to set it up properly so that it works on both platforms.

@knocte

This comment has been minimized.

Copy link

knocte commented Jul 1, 2012

Hey Rob, can you test by replacing it simply to "/"? I think normal slashes work nowadays in a crossplatform way.

@robpvn

This comment has been minimized.

Copy link
Contributor

robpvn commented Jul 1, 2012

Well, that's exactly how I fixed the issue for me locally on Linux, but I don't have any suitable Windows setups available right now, so I can't test if it works on Windows. :-/

@knocte

This comment has been minimized.

Copy link

knocte commented Jul 17, 2012

Hey Rob, I have a windows VM on my office-laptop, so I can test for you. I may have some time this weekend. Where can I find you in IRC so you help me set up to test it?

@robpvn

This comment has been minimized.

Copy link
Contributor

robpvn commented Jul 29, 2012

I'm terribly sorry, I must have missed this message from GitHub. I'm not normally terribly available on IRC unless I plan it ahead; so github messages might work just as well. Basically I believe I just followed the standard instructions in the readme, and any other experiences I posted in this blog post: http://www.robpvn.net/2012/06/30/hackfest-continues-experiences-with-cydin/ . I clicked through the intial setup thing, and let the folder name stay at the default, which turned out to be wrong. (And things go horribly wrong after that since it spews out exceptions every time it tries to find the Files folder)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment