Streamline build-packages-locally article#431
Conversation
b538e8c to
ad31d7e
Compare
This will make the article primarily present how to use sbuild, rather than suggesting multiple build tools. Instead, other build tools are linked at the very bottom. Known bug: copying the configuration file does not include the hashes in comments.
ad31d7e to
57180c6
Compare
|
Force pushed for typo correction |
|
@rkratky Nice, thanks! I appreciate the changes. Let me have a look. |
Main changes:
Removed numbers from headings
Gerunds in sub-headings
Long options for readability
More direct/authoritative style
Some formatting modifications
(Removed hard line breaks, but only because it was easier to edit like that. Otherwise it's not a problem.)
|
Merged your changes now, the looked good to me. |
|
You need to fix #270 if you're going to recommend unshare and ~/.config/sbuild/sbuild.pl` since otherwise the instructions won't work with the current Ubuntu LTS. See my notes from #295 I don't think enabling experimental tags ( The purge config is unnecessary for unshare, but it is helpful if you're going to use schroot for cross building. $build_dir = '/home/my_user/schroot/build';
$log_dir = '/home/my_user/schroot/logs';I don't think this should mention |
|
@jbicha Great points! Thanks for the feedback. It's really easy to assume everything from the perspective of the bleeding edge release, however that also risks these incompatibilities you mentioned here. In this case, I think the way forward is fixing #270 as you suggested, saying in |
mwhudson
left a comment
There was a problem hiding this comment.
Here's a few unsolicited comments! I think the changes mostly go in the right direction but some things are a bit muddled. It probably makes sense to document the new shiny unshare backend and then have a section that says "on older releases set up schroot like a b c"? (I think it's just a touch premature to drop all documentation of the schroot based solution).
|
With this $chroot_mode = 'unshare';
$unshare_mmdebstrap_keep_tarball = 1;
# /tmp is tmpfs and large builds will fail after filling all memory.
$unshare_tmpdir_template = '/var/tmp/tmp.sbuild.XXXXXXXXXX';
$clean_source = 0;
$run_lintian = 0; |
Co-authored-by: Robert Krátký <robert.kratky@canonical.com>
|
Hey folks! In order to get things moving, we are planning to get this merged and move the current unresolved comments into a separate issue in order to keep track of them. I will link the new issue here once it has been created. |
|
Here is the new issue: #536 For ease of use, I have linked the unresolved comments as well in there. |
Description
This pull request will streamline the article "How to build packages locally" by making it propose one recommended and more authoritative way of building packages using
sbuild, rather than multiple. It is needed to provide a more up to date and direct instruction set that should allow contributor's to build packages in a more uniform way.Related issue
Checklist
Additional notes (optional)
Configuration files as code blocks do not allow copying hashes ("#"), which is currently a UX bug since the text has to be manually copied instead.