search

Geonovum / technisch-register

1 stars 2 forks source link

Code geschikt gemaakt voor installeren op windows via git Bash #47

Closed ghost closed 8 years ago

ghost commented 8 years ago

git Bash kon niet overweg met backslashes die meekomen vanuit windows. Code aangepast zodat eventuele backslashes worden vervangen door forward slashes.

ndkv commented 8 years ago

Vreemd. Weet je waar de backslashes vandaan komen? Welke foutmelding krijg je?

Welke slashes gebruik je in settings.py? Daar komen de paden in principe vandaan.

De Git Bash is in principe een volwaardige Linux omgeving, de paden moeten daarom forward slashes gebruiken.

Overigens breekt 1 van de checks. Klik op Details om de foutmelding in Travis te zien. Volgens mij mist er een haakje op lijn 68 van backend.py

ghost commented 8 years ago

Hmm, ik heb bijna overal gewoon forward slashes gebruikt in settings.py.

Maar ik bedenk me nu wel dat ik bij _build_path = 'build' had staan. Na deze te hebben vervangen door build_path = '_build/' lukt het wel en is toevoegen van .replace('\', '/') niet meer nodig.

Dus de toevoegingen .replace('\', '/') in de code zijn dan inderdaad overbodig. Maar laat wel zien dat bij runnen op een windows machine het belangrijk is dat alle paden in settings.py met een forward slashes eindigen. Wanneer ik toch de readme aanpas ga ik dit er gelijk ook even in zetten.

Voor de duidelijkheid nog even de betreffende traceback:

_Traceback (most recent call last): File "initialize.py", line 30, in

build(s.sources_path, s.register_path, root_fs, repo['id']) File "C:\technisch-register\technisch_register\backend.py", line 41, in build build_folders(source, build_dir, standards_id[initiator], root, standards_id[initia tor]['cluster'], build_path) File "C:\technisch-register\technisch_register\backend.py", line 68, in build_folders if (root.exists(ospath.join(build_path, destination_temp, artifact))) == False: File "C:\Program Files (x86)\Python2.7.8\lib\site-packages\fs\errors.py", line 257, i n wrapper return func(self,_args,*_kwds) File "C:\Program Files (x86)\Python2.7.8\lib\site-packages\fs\osfs__init__.py", line 246, in exists return _exists(self.getsyspath(path)) File "C:\Program Files (x86)\Python2.7.8\lib\site-packages\fs\osfs__init__.py", line 166, in getsyspath self.validatepath(path) File "C:\Program Files (x86)\Python2.7.8\lib\site-packages\fs\base.py", line 305, in validatepath raise InvalidCharsInPathError(path)fs.errors.InvalidCharsInPathError: Path contains invalid characters: _build\register\gm lapplicatieschema_ Op 4 februari 2016 22:06 schreef Simeon Nedkov notifications@github.com: > Vreemd. Weet je waar de backslashes vandaan komen? Welke foutmelding krijg > je? > > Welke slashes gebruik je in settings.py? Daar komen de paden in principe > vandaan. > > De Git Bash is in principe een volwaardige Linux omgeving, de paden moeten > daarom forward slashes gebruiken. > > Overigens breekt 1 van de checks. Klik op Details om de foutmelding in > Travis te zien. Volgens mij mist er een haakje op lijn 68 van backend.py > > — > Reply to this email directly or view it on GitHub > https://github.com/Geonovum/technisch-register/pull/47#issuecomment-180048847 > .