Skip to content
This repository has been archived by the owner on Feb 16, 2024. It is now read-only.

build-source: Add support for debian/watch and depricate ubports.source_location #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mariogrip
Copy link
Member

@mariogrip mariogrip commented Dec 16, 2021

This adds support for using debian/watch files as the method to download
sources. This is a better way as it will automatically find and download
the version defined by changelog. This is also the official debian way
of doing this, this simplifies our merge with salsa debian folders. This
also adds support for checking for newer upstream versions if needed.

This also prints a deprecation warning when using source format lower
then 3.0 and when using ubports.source_location

…ce_location

This adds support for using debian/watch files as the method to download
sources. This is a better way as it will automatically find and download
the version defined by changelog. This is also the official debian way
of doing this, this simplifies our merge with salsa debian folders. This
also adds support for checking for newer upstream versions if needed.

This also prints a deprecation warning when using source location lower
then 3.0 and when using ubports.source_location
@mariogrip
Copy link
Member Author

Also ping @sunweaver for input

Copy link

@sunweaver sunweaver left a comment

Choose a reason for hiding this comment

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

See inline comments for my feedback.

build-source.sh Outdated
source_location_file=$(sourcedebian_or_source ubports.source_location)
if [ -n "$source_location_file" ]; then
echo "WARNING: ubports.source_location file is depricated, please use debian/watch file"

Choose a reason for hiding this comment

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

s/depricated/deprecated/g

build-source.sh Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants