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

[BUG] Deb assemble does not correctly show found deb artifact #4532

Closed
Tracked by #4087
peterzhuamazon opened this issue Mar 14, 2024 · 1 comment · Fixed by #4673
Closed
Tracked by #4087

[BUG] Deb assemble does not correctly show found deb artifact #4532

peterzhuamazon opened this issue Mar 14, 2024 · 1 comment · Fixed by #4673
Assignees

Comments

@peterzhuamazon
Copy link
Member

This seems like a print error rather than actual assemble error Found deb file: None:

2024-03-14 17:05:15 INFO     Found deb file: None
2024-03-14 17:05:18 INFO     Published /home/ci-runner/opensearch-build-peterzhuamazon/deb/dist/opensearch/opensearch-2.12.0-linux-x64.deb.
2024-03-14 17:05:18 INFO     Removing /tmp/tmpiqagb_xh
2024-03-14 17:05:19 INFO     Done.

Not sure if rpm is having the same issue.

Thanks.

@peterzhuamazon peterzhuamazon added bug Something isn't working good first issue Good for newcomers deb labels Mar 14, 2024
@github-actions github-actions bot added the untriaged Issues that have not yet been triaged label Mar 14, 2024
@rishabh6788 rishabh6788 removed the untriaged Issues that have not yet been triaged label Apr 1, 2024
@peterzhuamazon peterzhuamazon self-assigned this May 2, 2024
@peterzhuamazon
Copy link
Member Author

Apparently when debmake calls debuild to create the .deb package, if the source location is path/to/sourcedir, the the deb package will be created one level before, such as path/to.

make[1]: Entering directory '/tmp/tmp4cln0vxx'
dh_builddeb -- -Zgzip
dpkg-deb: building package 'opensearch' in '../opensearch_2.14.0_amd64.deb'.

This means the package now shows up in /tmp instead.
And the initial commit did not specify the actual location leaving it as blank.

Thanks.

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