-
Notifications
You must be signed in to change notification settings - Fork 181
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
Update SETUP.md to replace the correct random string #182
Conversation
Something weird is happening with
|
Possible we may need |
Also possible that |
@dhimmel Have you seen this Travis error before? |
No.
We don't run Lines 7 to 17 in 2a4c813
I'll have more time to look tomorrow, but I've restarted the build in case it's temporary. Otherwise, perhaps we can add a workaround, as suggested in the other issues, to this PR. |
Thanks. Also failing on a clone of this repository I made last night for a new manuscript, so it's not unique. |
Thanks. Also failing on a clone of this repository I made last night for a new manuscript, so it's not unique. This is bad. Perhaps we want to enable daily cron builds of rootstock so we can more quickly discover and pinpoint these issues. This is disappointing because we have tried to pin as much as possible regarding our conda environment. However, the environment is still changing (and breaking). A good reason for us to switch to docker where CI runs don't have to rebuild the env each time. |
> conda config: error: argument --prepend/--add: not allowed with argument --set
Turned out installing conda's precompiled |
This build is based on 717630e. This commit was created by the following Travis CI build and job: https://travis-ci.com/manubot/rootstock/builds/102622378 https://travis-ci.com/manubot/rootstock/jobs/181210990 [ci skip] The full commit message that triggered this build is copied below: Fix SETUP.md random string & psutil install Merges #182 Update SETUP.md to replace the correct random string Fix missing x86_64-conda_cos6-linux-gnu-gcc in psutil install by installing psutil using conda. .travis.yml: one configuration option per line (non-functional whitespace change)
This build is based on 717630e. This commit was created by the following Travis CI build and job: https://travis-ci.com/manubot/rootstock/builds/102622378 https://travis-ci.com/manubot/rootstock/jobs/181210990 [ci skip] The full commit message that triggered this build is copied below: Fix SETUP.md random string & psutil install Merges #182 Update SETUP.md to replace the correct random string Fix missing x86_64-conda_cos6-linux-gnu-gcc in psutil install by installing psutil using conda. .travis.yml: one configuration option per line (non-functional whitespace change)
It would be nice if there was a way to use conda to help us locate which versions of the unpinned packaged had changed between builds. In this case, Docker may indeed be the best solution here. I expect we would still want to maintain a functional conda environment but could use Docker for reliable builds. |
I'm now realizing that it may not be as simple as the version of a single package. Other manubot builds are still passing even with |
I've used Docker a few times in the past, but not within the past year or so. Is there a simple recommended way for package maintainers to effectively say to users: "Here is our Docker image, to use it execute this one line"? Or is it still a multistep process with potential permissions issues (still need |
@slochower I am no Docker expert, but in my limited experience I'd say that yes it easy to do that if you already have Docker configured on your system.
I haven't used Singularity much. I too have seen a lot of clusters migrating to Singularity. My understanding is that Singularity can run Docker containers so there is no major downside to creating a Docker container for Singularity or Docker users. |
Merges manubot/rootstock#182 Update SETUP.md to replace the correct random string Fix missing x86_64-conda_cos6-linux-gnu-gcc in psutil install by installing psutil using conda. .travis.yml: one configuration option per line (non-functional whitespace change)
Merges manubot/rootstock#182 Update SETUP.md to replace the correct random string Fix missing x86_64-conda_cos6-linux-gnu-gcc in psutil install by installing psutil using conda. .travis.yml: one configuration option per line (non-functional whitespace change)
The
sed
command inSETUP.md
was not using the appropriate random string indeploy.sh
.