-
Notifications
You must be signed in to change notification settings - Fork 483
* Add arbitrary build param support #174
base: master
Are you sure you want to change the base?
* Add arbitrary build param support #174
Conversation
|
Any update on this PR? |
|
Merging this PR should solve #202 as well. |
| @Parameter(property = "dockerfile.buildArgs") | ||
| private Map<String,String> buildArgs; | ||
|
|
||
| @Parameter(property = "dockerfile.builParams") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling is wrong. "dockerfile.buildParams"
bcd3100 to
21f2206
Compare
21f2206 to
a390ec0
Compare
|
fixed the typo |
|
Hi @DylanSchell, @steven-sheehy |
|
Any chance this can get the conflict fixed and merged? I am really needing this to set build arguments like network=host |
|
We use intermediate containers to save space. This option would help a lot for CI builds. I could use |
|
I could really do with this PR... I need to add some options to docker build command... |
|
Can't you just build the whole file from scratch?
…On Tue, Mar 22, 2022 at 4:19 AM Sergio Eduardo Dantas de Oliveira < ***@***.***> wrote:
I could really do with this PR... I need to add some options to docker
build command...
—
Reply to this email directly, view it on GitHub
<#174 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUTIGD6X7SSGDW3C2OBDWBLVBENZ5ANCNFSM4E7VRENA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
For some builds, we need shm-size support. by implementing arbitrary build parameter support, it should be possible to just configure
1000000000