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

Packaging and template cleanups #109

Merged
merged 5 commits into from Feb 7, 2019
Merged

Conversation

guillemj
Copy link
Contributor

@guillemj guillemj commented Feb 7, 2019

This MR contains several packaging and template cleanups and improvements. Even though some of these can be considered stylistic changes, they serve specific practical purposes, as documented in their respective commits.

By default the build directory is arch-qualified, which means that we
cannot easily ignore them in places like .gitignore. Use a well-known
pathname for this.
Format fields in a way that minimizes diff output and that makes it
easier to copy & paste or move lines around between fields:

 - Adding a trailing comma on the last entry makes appending new
   entries not affect the last one.
 - Starting the field value on the next line makes inserting or changing
   the first value generate less diff.
 - Indenting by one space, makes the contents movable between fields,
   and gives a unified format to all such fields.
 - Sorting makes the contents easier to find and easier to avoid
   adding duplicates.

We cannot currently do this for the Go-Import-Path field, because
dh-golang does not cope with this file being multi-line with an empty
first line, yet.
This makes it possible to build packages w/o requiring (pseudo-)root
privileges.
…Indep

Not all build dependencies are required all the time. Move those into
the fields that are appropriate for their type.
@stapelberg stapelberg merged commit 90415f4 into Debian:master Feb 7, 2019
@stapelberg
Copy link
Collaborator

Thanks!

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

2 participants