Closed vmiheer closed 5 years ago
This looks like a really great notice to add. It seems to be true and the compilation difficulties are otherwise difficult to debug. Unfortunately, the ngstools
repo no longer exists.
@vmiheer : Is there any way to change the title of this pull request to make it more informative? If not, perhaps we should close the request and build a new one?
@r-barnes, does this look okay? Otherwise please suggest alternative I can modify 😃 .
@vmiheer : This looks much better. I think the only other problem is that we have your continuous integration commit on this PR. Ideally, we should build a separate branch on our repo and remove that specific change, and any others which wouldn't immediately translate to Nvidia, though this is the only one I can think of. That branch, which we could call upstream_pr
or similar would then be the one we'd use for the PR.
@vmiheer : Actually, could you close this pull request? I'll make a new one from upstream_pull_request
.
Nvidia is changing who owns the NvBio project and it turns out I know the new owner, so it might streamline the pull if I'm able to manage it.
Also add alternate repo.