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

Add an <internet/> kind to requires/recommends/supports /3 #426

Merged
merged 2 commits into from Aug 22, 2022

Conversation

ximion
Copy link
Owner

@ximion ximion commented Aug 22, 2022

This is a version of #425 with some minor fixes and added validation.

pwithnall and others added 2 commits August 22, 2022 12:30
This allows apps to specify whether they require the internet to work,
or whether they explicitly work fully without internet.

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
Signed-off-by: Matthias Klumpp <matthias@tenstral.net>

Fixes: #343
@ximion ximion merged commit aae82e4 into master Aug 22, 2022
15 checks passed
@ximion ximion deleted the wip/mak/rel_internet branch August 22, 2022 10:37
@JakobDev
Copy link
Contributor

Maybe you should also add a example for offline-only, so they User know in which relations-tag he should put it.

@ximion
Copy link
Owner Author

ximion commented Aug 22, 2022

This is shaping up to be the most extensive example block in the whole specification :-D - I normally like to keep the examples quite brief, as information should be derivable from the text, but I also do see the point here of adding an example for it in a supports block for even more clarity.

@JakobDev
Copy link
Contributor

I found this in the documentation as exmaple:

<!-- never uses the internet, even if available -->
<requires>
  <internet>never</internet>
</requires>

This is invalid and should be offline-only.

@ximion
Copy link
Owner Author

ximion commented Aug 22, 2022

Ha! Well spotted! Thanks for having another look, it's good to not make a release with these obvious but very confusing errors (and I didn't spot that when reviewing the original PR!)

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.

None yet

3 participants