jackyf / cupt

high-level package manager for Debian, inspired from APT
36 stars 4 forks source link

sources.list: non-space-surrounded braces for options not recognised #10

Closed UriHerrera closed 5 years ago

UriHerrera commented 5 years ago

cupt gives an error message when a repository is specifying an architecture in its configuration file and results in cupt skipping the repository. e.g., this is the output from the Google Chrome repository.

E: no download handlers defined for the protocol '[arch=amd64]/dists/http'
E: no download handlers defined for the protocol '[arch=amd64]/dists/http'
W: failed to download (In)Release for '[arch=amd64] http://dl.google.com/linux/chrome/deb//'
W: failed to download (In)Release for '[arch=amd64] http://dl.google.com/linux/chrome/deb//'
E: there were errors while downloading release and index data
E: error performing the command 'update'

If I remove the line specifying the architecture cupt doesn't give any warning or error message.

jackyf commented 5 years ago

Thank you for the report.

There are two issues here: 1) not parsing the option correctly, due to my overly strict interpretation of the option format. I'll fix that. The workaround: use [ arch=amd64 ] (that is, insert a space the opening bracket and a space before the closing bracket). 2) libcupt does not support "MultiArch". Even if the arch=amd64 is parsed correctly, it will be still ignored. If your system requires installing packages for non-native architectures, I'm afraid cupt as of today will not be able to help.

jackyf commented 5 years ago

This (only the syntax part, as per updated title) is now fixed in the master branch.

UriHerrera commented 5 years ago

Thanks for the fix :)