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

Support source parts directory #119

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Conversation

colincasey
Copy link
Contributor

Add support for copying the source list configuration files from /etc/apt/sources.list.d into the sources used for the apt-get command line invocations and makes sure these are included in the APT_OPTIONS.

GUS-W-14674577

Fixes #114

@colincasey colincasey requested a review from a team as a code owner March 19, 2024 14:23
@edmorley edmorley requested review from dzuelke and removed request for a team March 19, 2024 19:59
bin/compile Show resolved Hide resolved
@colincasey colincasey force-pushed the support_source_parts_dir branch 2 times, most recently from 2d5f283 to d2897fc Compare March 20, 2024 17:07
@colincasey colincasey changed the base branch from fix_empty_aptfile to fix_for_aptfile_with_no_packages March 20, 2024 17:07
@colincasey colincasey force-pushed the support_source_parts_dir branch from d2897fc to 5468a46 Compare March 20, 2024 17:09
@colincasey colincasey mentioned this pull request Mar 20, 2024
@colincasey colincasey force-pushed the support_source_parts_dir branch from 5468a46 to 9e1a71e Compare March 20, 2024 17:24
@colincasey colincasey force-pushed the fix_for_aptfile_with_no_packages branch from e8b841d to e5a642c Compare March 22, 2024 15:48
@colincasey colincasey force-pushed the support_source_parts_dir branch from 9e1a71e to 3b066dc Compare March 22, 2024 15:48
Copy link
Contributor

@dzuelke dzuelke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use jammy builds for noble in the tests:

root@3ecdc55e4611:~# wget -q https://github.com/wkhtmltopdf/packaging/releases/download/0.12.6.1-2/wkhtmltox_0.12.6.1-2.jammy_amd64.deb
root@3ecdc55e4611:~# dpkg -x wkhtmltox_0.12.6.1-2.jammy_amd64.deb .
root@3ecdc55e4611:~# find usr -type f \( -executable -o -name "*.a" \) -exec ldd -r "{}" \; | grep -e "not found" -e "undefined symbol" || echo "all good"
all good

@colincasey colincasey force-pushed the fix_for_aptfile_with_no_packages branch from e5a642c to 2b1891a Compare March 28, 2024 13:52
Base automatically changed from fix_for_aptfile_with_no_packages to master March 28, 2024 13:53
@colincasey colincasey force-pushed the support_source_parts_dir branch from 3b066dc to 55ef1b8 Compare March 28, 2024 13:53
@colincasey colincasey force-pushed the support_source_parts_dir branch from 55ef1b8 to aa5c018 Compare March 28, 2024 14:23
@colincasey colincasey requested a review from dzuelke March 28, 2024 14:26
@colincasey
Copy link
Contributor Author

colincasey commented Mar 28, 2024

@dzuelke I've added your suggestion to use the jammy package for noble in custom package url tests.

@colincasey colincasey merged commit f95568b into master Mar 28, 2024
5 checks passed
@colincasey colincasey deleted the support_source_parts_dir branch March 28, 2024 17:57
@colincasey colincasey mentioned this pull request Mar 28, 2024
Frzk pushed a commit to Scalingo/apt-buildpack that referenced this pull request May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Heroku-24 compatibility (support sources in /etc/apt/sources.list.d/)
3 participants