Skip to content

Add more platforms to the setup #1362

Closed

Description

@Fohlen

Heya guys.
At @inexorgame we are starting to rely heavily on nodegit
Is there any way that more platforms will be supported in the future by nodegit?
As far as I can tell, Docker could be used with a set of base images just to do that easily.

Activity

  1. rcjsuen commented on Aug 30, 2017

    @rcjsuen
    Member

    @Fohlen When you say "more platforms", what are you referring to exactly? More operating systems (say NetBSD and OpenBSD)? More Linux distributions? More versions of existing supported operating systems?

    How can Docker images be used to easily do this?

  2. saper commented on Aug 30, 2017

    @saper
    Collaborator

    @Fohlen Docker will not help whenever a different kernel is required.

  3. Fohlen commented on Aug 31, 2017

    @Fohlen
    Author

    Firstly, I apologize for the rather vague ticket description.

    @rcjsuen with more platforms, in specifically, I imagine:

    • more versions of Node.js (newer ones, if possible)
    • more versions of npm (the 5 series in particular)
    • more linux distributions, such as Cent OS and Arch Linux
    • the latest version of OSX (since it is building manually on Mac currently)
    • Windows 8 & 10 (AppVeyor)

    Over at @inexorgame we use a combination of traditional bash scripts and Docker with different base images to build the same thing under different platforms. This approach may be comfortable for nodegit, or may not.

    Commenting on @saper, this is true. OpenBSD/NetBSD might be needed to be build on some private Jenkins, if there is enough user demand.

    If necessary, I think we can also add some templates in a PR, maybe they come in handy.

  4. rcjsuen commented on Aug 31, 2017

    @rcjsuen
    Member

    @Fohlen I guess when you say "the setup" you mean for the NodeGit binaries to be available via installation from npm without having to manually compile NodeGit, correct?

    From looking at the latest builds, we do currently build on Node 8.4.0 and npm 5.3.0 on Travis CI and Node 8.3.0 and npm 5.3.0 on AppVeyor. It seems like Travis CI is building on OS X El Captain (and not macOS Sierra). It seems you have to either use Windows Server 2012 R2 or Windows Server 2016 on AppVeyor so there's not much we can do there I think?

    As for Cent OS and Arch Linux, there are several open issues about those two (as well as other) Linux distributions so the demand is certainly there.

  5. rsmarples commented on Sep 29, 2017

    @rsmarples
    Contributor

    There are some outstanding pull requests for better BSD platform support - namely #1237, #1344 and #1374.

    With these changes, nodegit compiles at least on modern BSD platforms and I can vouch for NetBSD.

  6. Fohlen commented on Oct 22, 2017

    @Fohlen
    Author

    I have to report that according to our CI only

    • Node 6.11.4, 7.10.1 on Ubuntu Trusty, OSX and Windows are supported.
      We tested a couple more of platforms but as it seems currently Node >7 and other distros than Trusty are unsupported.
  7. Fohlen commented on Nov 5, 2017

    @Fohlen
    Author

    https://github.com/nodegit/nodegit/blob/master/.travis.yml#L17

    What's the issue here to:

    • support the actual latest versions of Node 6 and 7 or support even more platforms (7.3? wtf?)
    • if you're concerned about your AWS storage, you should consider moving your binaries to bintray. They offer up to 1 TB for open source projects
    • additionally:
      • you should use Docker to build (!at least) for Ubuntu Xenial. Seriously Trusty Tahr is no more platform to target end users (maybe servers, but not end users)

    As for the Windows and OSX support, there should be more node versions covered as well. Anyhow, nvm and windows-nvm can easily be used here..

  8. Croydon commented on Jul 27, 2018

    @Croydon
    Contributor

    I guess this can be closed in favor of #1298

  9. Fohlen commented on Jan 15, 2019

    @Fohlen
    Author

    I will be closing this. It is insane that such a huge project does not build for the LTS version of node.

  10. rcjsuen commented on Jan 15, 2019

    @rcjsuen
    Member

    @Fohlen NodeGit currently builts for Node.js v6, v8, and v10.

    nodegit/.travis.yml

    Lines 23 to 26 in 1bc53bf

    node_js:
    - "10"
    - "8"
    - "6"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions