Skip to content

Conversation

@imobachgs
Copy link
Contributor

@imobachgs imobachgs commented Mar 20, 2025

Problem

YaST-like URLs are not supported for info files because we use CURL to retrieve the file.

Fixes #2176.

Solution

Use "agama download" to get the info file.

Testing

  • Updated the integration test.

Warning

The tests fail because the agama download command tries to connect to the Agama HTTP server even if unnecessary. See #2192 for a fix.

@imobachgs imobachgs requested a review from dgdavid April 2, 2025 05:48
@imobachgs

This comment was marked as resolved.

@dgdavid

This comment was marked as outdated.

@imobachgs imobachgs merged commit 0384831 into master Apr 2, 2025
16 checks passed
@imobachgs imobachgs deleted the agama-download branch April 2, 2025 08:01
@imobachgs imobachgs mentioned this pull request Apr 22, 2025
imobachgs added a commit that referenced this pull request Apr 22, 2025
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.

inst.info should use agama download instead of curl

3 participants