slluis/cydin

DataPath DefaultValue is Windows-specific

Opened this issue · 4 comments

robpvn commented

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 commented

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

robpvn commented

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 commented

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 commented

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)