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

reformat package lists to put one item per line, sort lists #206

Closed
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@sneak

sneak commented Sep 19, 2018

this makes it much easier to see what is or isn't installed (and makes diffs simpler)

sneak added some commits Sep 19, 2018

lots of changes, see summary
* default to ubuntu default keyboard layout, don't override with gb
* run rc.local.txt from /boot if exists
* don't export stage4/5 images or noobs
* export stage2 -keyternal image
* don't resize root partition on first boot
* don't enable swap by default

@sneak sneak force-pushed the keyternal:master branch from fc08f4e to c217a5e Oct 15, 2018

@XECDesign

This comment has been minimized.

Contributor

XECDesign commented Oct 15, 2018

If the packages need to be more organised, I'd rather they be categorised rather than just sorted alphabetically. There are packages which should exist together, but if they're far apart in the file, it will be easy to accidentally miss them.

sneak added some commits Oct 16, 2018

@XECDesign

This comment has been minimized.

Contributor

XECDesign commented Oct 16, 2018

Looks like you used your master branch for this PR, so all of your other changes are being added here.

@sneak

This comment has been minimized.

sneak commented Oct 16, 2018

You are absolutely correct and I'm sorry about that! Will fix.

@sneak

This comment has been minimized.

sneak commented Oct 16, 2018

Actually if a simple alphabetic listing isn't suitable, then I'm not going to redo the PR. That's two simple and common-sense PRs that you've rejected outright now so my contributions end here. I'll just maintain my own fork.

@sneak sneak closed this Oct 16, 2018

@sneak sneak referenced this pull request Oct 16, 2018

Closed

packages: 1 per line, sorted #213

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment