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

Windows development fixes #1585

Merged
merged 2 commits into from Nov 25, 2020
Merged

Windows development fixes #1585

merged 2 commits into from Nov 25, 2020

Conversation

dopplershift
Copy link
Member

Description Of Changes

Fixes a couple things I ran into while trying to debug on Windows the other day:

  • Ensure hashes for our test data files match on Windows by enforcing line endings
  • Fix version numbering so that local builds appear as such--using "dirty tag" only appends "+dirty" if files have been changed, otherwise dev builds from git look like "1.0.0rc2.post68". This can lead to pooch trying to download static data from a tag matching it rather than master.

Checklist

This prevents git from changing the line endings on these files on
Windows, which would change the hash and cause pooch to download the
data files.
By using "dirty-tag", it disables the +gHASH part of the version, which
leads to our development versions appearing as a non-local version to
pooch. This can lead to development snapshots trying to download from
non-existent versions instead of master.
@dopplershift dopplershift added Area: Infrastructure Pertains to project infrastructure (e.g. CI, linting) Type: Bug Something is not working like it should labels Nov 25, 2020
@dopplershift dopplershift added this to the 1.0 milestone Nov 25, 2020
@dcamron dcamron merged commit e4ea919 into Unidata:master Nov 25, 2020
@dopplershift dopplershift deleted the win-fixes branch November 25, 2020 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Infrastructure Pertains to project infrastructure (e.g. CI, linting) Type: Bug Something is not working like it should
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants