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.

Sign up for GitHub

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

Jump to bottom

Non-zero exit code but package get's build and uploaded successfully #836

Open
timosachsenberg opened this issue Feb 9, 2023 · 0 comments
Open

Non-zero exit code but package get's build and uploaded successfully #836

timosachsenberg opened this issue Feb 9, 2023 · 0 comments

Comments

@timosachsenberg
Copy link

Hi, our bioconda build always returns with exit code != 0 because it doesn't find the tar.gz file

OpenMS/OpenMS#6514 (comment)

We disabled upload during build and the perform the upload separately - which works:

Using Anaconda API: https://api.anaconda.org
[DEBUG] Using token from command line args
Using "openms" as upload username
Processing '/home/builder/jenkins-ws/ws/openms/ntly/conda/output/linux-64/libopenms-3.0.0dev-h135471a_0.tar.bz2'
Detecting file type...
[DEBUG] Testing if conda package ..
[DEBUG] This is a conda package
File type is "conda"
Extracting conda package attributes for upload
Creating package "libopenms"
Creating release "3.0.0dev"
Uploading file "openms/libopenms/3.0.0dev/linux-64/libopenms-3.0.0dev-h135471a_0.tar.bz2"
[WARNING] Distribution "linux-64/libopenms-3.0.0dev-h135471a_0.tar.bz2" already exists. Removing.
Upload complete

Could this be an error in build?
jpfeuffer added a commit to jpfeuffer/bioconda-utils that referenced this issue Feb 24, 2023
@jpfeuffer
fix bioconda#836
fb369fc
bgruening added a commit that referenced this issue Nov 19, 2024
@jpfeuffer @daler
@johanneskoester @bgruening
fix: success check does not consider pkg_dir #836 (#838)
6240d31 
completely untested but it is an obvious bug.
If you want to change the package dirs earlier in the code somewhere,
please suggest changes

---------

Co-authored-by: Ryan Dale <[email protected]>
Co-authored-by: Johannes Köster <[email protected]>
Co-authored-by: Johannes Köster <[email protected]>
Co-authored-by: Björn Grüning <[email protected]>
bgruening pushed a commit that referenced this issue Nov 30, 2024
@github-actions
chore: release 3.5.0 (#1020)
5910a10 
🤖 I have created a release \*beep\* \*boop\*
---
##
[3.5.0](https://www.github.com/bioconda/bioconda-utils/compare/v3.4.1...v3.5.0)
(2024-11-29)


### Features

* add rust to should_use_compilers lint
([#1018](https://www.github.com/bioconda/bioconda-utils/issues/1018))
([0dacce9](https://www.github.com/bioconda/bioconda-utils/commit/0dacce91865e649061b7d18008c34789287c65fa))


### Bug Fixes

* success check does not consider pkg_dir
[#836](https://www.github.com/bioconda/bioconda-utils/issues/836)
([#838](https://www.github.com/bioconda/bioconda-utils/issues/838))
([6240d31](https://www.github.com/bioconda/bioconda-utils/commit/6240d31e706b91b4b217ab10305fb84aabd8a147))
---


This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@timosachsenberg