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

Fix v3.6.x (and above) download issues for Mac OSX, Windows #35

Merged

Conversation

@cantremember
Copy link

@cantremember cantremember commented Oct 15, 2018

Fixes #33

  • 'osx' archive naming changed in v3.5
  • 'win32' archive naming changed in v3.5
  • not all signatures end with trailing whitespace
  • MongoDB site Downloads send folks to 'fastdl.mongodb.org'
  • added Test Suite for package/arch/version/os variants
  • TODOs for 'linux'; 'legacy', 'static', 'debian92', 'rhel57'
  • TypeScript development tooling w/ npm run build

negotiable: Test Suite coverage

to ensure that i produced a solution that really worked, i added an extensive data-driven Test Suite. it's great for proving out that the package works ... however it's a time sink and its scope of coverage could be reduced

feedback is welcome

negotiable: TODOs

i noticed the following 'operating system' namespaces when looking at all the download options for 'linux':

  • 'debian92' and 'rhel57', which are real OS versions for linux that aren't yet flushed out
  • 'legacy', which may or may not be related to the "using legacy release" notes in the current code
  • 'static', the meaning of which i do not know

i can remove any or all of those TODO comments from this commit, at your discretion

negotiable: TypeScript tooling

if you'd rather there wasn't an npm script and that the devDependencies didn't bring in tsc,
i will happily remove those changes

i could also break them off into a separate PR

did not run tsc

trying to merge the results of npm run build will likely reduce spurious conflicts -- probably not a good thing for the Reviewer

#33

- 'osx' archive naming changed in v3.5
- 'win32' archive naming changed in v3.5
- not all signatures end with trailing whitespace
- MongoDB site Downloads send folks to 'fastdl.mongodb.org'
- added Test Suite for package/arch/version/os variants
- TODOs for 'linux'; 'legacy', 'static', 'debian92', 'rhel57'
- TypeScript development tooling w/ `npm run build`
@cantremember cantremember changed the title Fix v3.6.x (and above) download issues for Mac OSX Fix v3.6.x (and above) download issues for Mac OSX, Windows Oct 15, 2018
Copy link
Contributor

@LoneRifle LoneRifle left a comment

lgtm otherwise, thanks for the PR!

src/mongodb-download.ts Outdated Show resolved Hide resolved
@cantremember
Copy link
Author

@cantremember cantremember commented Nov 29, 2018

you are very welcome, @LoneRifle 👍

@LoneRifle
Copy link
Contributor

@LoneRifle LoneRifle commented Nov 30, 2018

I've not gotten the chance to test this on my machine, but once I do I'll merge to master and have one additional change to support Ubuntu 18 before kicking it out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants