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

Add fossil support #224

Closed
wants to merge 2 commits into from
Closed

Add fossil support #224

wants to merge 2 commits into from

Conversation

dilyn-corner
Copy link
Contributor

Mostly just some cut and paste to segregate git to its own functions (one for package cloning, one for repository updates) and adding an analogous function for fossil.

I've briefly tested it -- it updates repositories and prompts for updates, it clones repositories/branches/commits and builds packages.

We could add more things to this or simplify it some if you would like!

Dilyn Corner added 2 commits July 9, 2021 16:20
Turn pkg_extract() into a generic vcs nabber, move fossil+* and git+* to
their own functions (pkg_extract_vcs()).
Comment on lines +273 to +276
# Fossil repository
elif [ -z "${2##fossil+*}" ]; then
_res=$2

Copy link
Contributor Author

@dilyn-corner dilyn-corner Jul 10, 2021

Choose a reason for hiding this comment

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

We could probably consolidate this with the # Git repository elif above.

@dylanaraps
Copy link
Member

dylanaraps commented Jul 10, 2021

Support for additional version control software is not on my radar right now. I want to first refactor git support to cache clones and only pull the desired commit/difference then we can look at supporting fossil. This will happen later on though - my focus is on wayland/openssl for the time being.

I need to also take a look at fossil and try it out.

@dilyn-corner
Copy link
Contributor Author

Totally fine. Mostly just wanted to put this out there; I'm making the switch soon(tm) and figured I'd share the small tweak I'm introducing.

@dylanaraps
Copy link
Member

Given the plans to make updates generic (next after #261), I'm closing this.

@dylanaraps dylanaraps closed this Aug 25, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants