Skip to content

Support Go package paths & GBB_PATH#2588

Merged
hugelgupf merged 7 commits intou-root:mainfrom
hugelgupf:use-new-glob
Dec 29, 2022
Merged

Support Go package paths & GBB_PATH#2588
hugelgupf merged 7 commits intou-root:mainfrom
hugelgupf:use-new-glob

Conversation

@hugelgupf
Copy link
Copy Markdown
Member

@hugelgupf hugelgupf commented Dec 28, 2022

Support for Go package paths is back!

Support for Go package path globs is back.

New support for environment variable GBB_PATH.

README should explain everything.

@hugelgupf hugelgupf force-pushed the use-new-glob branch 4 times, most recently from f6a86ff to 3f3ada3 Compare December 29, 2022 02:17
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 29, 2022

Codecov Report

Base: 73.55% // Head: 73.54% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (f93bcef) compared to base (c9a228d).
Patch coverage: 85.71% of modified lines in pull request are covered.

❗ Current head f93bcef differs from pull request most recent head 60cb1e5. Consider uploading reports for the commit 60cb1e5 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2588      +/-   ##
==========================================
- Coverage   73.55%   73.54%   -0.01%     
==========================================
  Files         407      406       -1     
  Lines       41293    41209      -84     
==========================================
- Hits        30372    30309      -63     
+ Misses      10921    10900      -21     
Impacted Files Coverage Δ
pkg/testutil/testutil.go 18.75% <0.00%> (ø)
pkg/vmtest/integration.go 64.19% <71.42%> (-2.47%) ⬇️
pkg/uroot/builder/binary.go 87.50% <83.33%> (-8.16%) ⬇️
pkg/vmtest/gotest.go 66.66% <88.88%> (+2.94%) ⬆️
pkg/uroot/builder/gbb.go 52.77% <100.00%> (-3.64%) ⬇️
pkg/uroot/uroot.go 63.63% <100.00%> (-6.28%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

rminnich
rminnich previously approved these changes Dec 29, 2022
Copy link
Copy Markdown
Member

@rminnich rminnich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this works for me now

./u-root cmds/core/* ../cpu/cmds/cpu
this does too
export D=`pwd` && (cd /tmp && $D/u-root -uroot-source $D 'cmds/core/*')

@hugelgupf hugelgupf marked this pull request as ready for review December 29, 2022 07:18
@hugelgupf hugelgupf changed the title Use new glob from gobusybox Support Go package paths & GBB_PATH Dec 29, 2022
Signed-off-by: Chris Koch <[email protected]>
Allows resolving of both file system and Go package paths & globs;
introduces GBB_PATH support.

Signed-off-by: Chris Koch <[email protected]>
Signed-off-by: Chris Koch <[email protected]>
All uses have been replaced.

Signed-off-by: Chris Koch <[email protected]>
@rminnich
Copy link
Copy Markdown
Member

This is delightful to have back.

@hugelgupf hugelgupf merged commit 656d490 into u-root:main Dec 29, 2022
@hugelgupf hugelgupf deleted the use-new-glob branch December 29, 2022 21:25
You can now use the u-root command to build an initramfs. Here are some
examples with $UROOT_PATH being the path to where the u-root sources are on the disk
(explicitly specifiying this is only necessary if not runnig u-root inside the root of the repository):
You can now use the u-root command to build an initramfs. Here are some examples
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure why 'now' is in there, since the initramfs has been in there for-f-ing ever :-) Maybe I did that.

(explicitly specifiying this is only necessary if not runnig u-root inside the root of the repository):
You can now use the u-root command to build an initramfs. Here are some examples
with $UROOT_PATH being the path to where the u-root sources are on the disk
(explicitly specifiying this is only necessary if not runnig u-root inside the
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

specifying

no i after f

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.

2 participants