Skip to content

Small build fixes - #615

Closed
antisaling wants to merge 3 commits into
kworkflow:unstablefrom
antisaling:small-build-fixes
Closed

Small build fixes#615
antisaling wants to merge 3 commits into
kworkflow:unstablefrom
antisaling:small-build-fixes

Conversation

@antisaling

@antisaling antisaling commented May 26, 2022

Copy link
Copy Markdown
Contributor

Those are small fixes to the src/build.sh file I made during PR's #543 development. Note: it's based on #614.

@codecov-commenter

codecov-commenter commented May 26, 2022

Copy link
Copy Markdown

Codecov Report

Merging #615 (679f07d) into unstable (2e219fc) will decrease coverage by 0.01%.
The diff coverage is 85.18%.

❗ Current head 679f07d differs from pull request most recent head 0b88130. Consider uploading reports for the commit 0b88130 to get more accurate results

@@             Coverage Diff              @@
##           unstable     #615      +/-   ##
============================================
- Coverage     73.76%   73.74%   -0.02%     
============================================
  Files            36       36              
  Lines          5553     5558       +5     
============================================
+ Hits           4096     4099       +3     
- Misses         1457     1459       +2     
Impacted Files Coverage Δ
src/config_manager.sh 79.71% <83.33%> (-0.12%) ⬇️
src/build.sh 90.72% <85.00%> (-0.86%) ⬇️
src/signal_manager.sh 96.15% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e219fc...0b88130. Read the comment docs.

@antisaling
antisaling force-pushed the small-build-fixes branch 2 times, most recently from 939f11c to 0b88130 Compare May 26, 2022 23:20
@rodrigosiqueira

Copy link
Copy Markdown
Collaborator

@isinyaaa
Could you rebase this PR?

When used without a number, the `-j` flag will create as many threads as
needed. This works pretty well for small tasks such as when using one of
the various "menu"config utilities or when preparing an environment for
a new compilation.

Signed-off-by: Isabella Basso <isabbasso@riseup.net>
As per [1], the `kw_parse` wrapper should be used instead of the
`getopt` command for command line options parsing in kw. This commit
also improves readability of flags passed in the build test.

[1] - src: kwlib: Create command line parser for kw (48db7ec)

Signed-off-by: Isabella Basso <isabbasso@riseup.net>
This fixes whitespace incoherences that happen when some options are not
used when building the kernel.

Signed-off-by: Isabella Basso <isabbasso@riseup.net>
@antisaling
antisaling force-pushed the small-build-fixes branch from 0b88130 to d4ec86a Compare May 27, 2022 18:32
@rodrigosiqueira

Copy link
Copy Markdown
Collaborator

Merged!
Thanks a lot!

@antisaling
antisaling deleted the small-build-fixes branch May 27, 2022 19:39
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.

3 participants