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

get_git_changeset fails on windows #358

Closed
timlinux opened this issue Oct 19, 2012 · 5 comments
Closed

get_git_changeset fails on windows #358

timlinux opened this issue Oct 19, 2012 · 5 comments

Comments

@timlinux
Copy link
Contributor

Problem: When using a release type of 'alpha' a subprocess error is spawned when loading InaSAFE on Windows. I traced the problem back down to get_git_changeset.

Short term fix: Changing release state to beta or final solves the problem as these do not trigger a call to get_git_changeset

Long term fix: Figure out how to make this work sustainable on windows.

@ghost ghost assigned ingenieroariel Oct 19, 2012
@timlinux
Copy link
Contributor Author

Oh for clarity that issue is in safe.common.version.get_git_changeset()

@ismailsunni
Copy link
Member

@timlinux tim, I think this issue occur again due this commit : bd5356b. Can you fix this?

ismailsunni added a commit that referenced this issue Dec 4, 2012
@ismailsunni
Copy link
Member

Somehow, that approach is failed again. It worked before.

For short term fix, I will check if it's Windows or Mac, and return something else, i.e. 'dev'.

ismailsunni added a commit that referenced this issue Jan 3, 2013
@timlinux timlinux mentioned this issue Feb 7, 2013
@timlinux
Copy link
Contributor Author

timlinux commented Nov 7, 2014

Can you confirm if your fix is in place @ismailsunni and close this out if it is?

ismailsunni added a commit to ismailsunni/inasafe that referenced this issue Nov 13, 2014
@ismailsunni ismailsunni mentioned this issue Nov 13, 2014
@ismailsunni
Copy link
Member

Hi @timlinux
I made some unit tests for this fix, and made a PR of it on #1349

I can confirm that it's fixed. Tested on Windows 7 and Ubuntu 14.04.

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

3 participants