-
Notifications
You must be signed in to change notification settings - Fork 129
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
feat: new ice build'' #458
Conversation
cf0bc93
to
4fff95e
Compare
I've also added null'' ice to be automatically triggered from build''. There is also |
@vladdoster: Why this PR isn't merged yet? It adds a pull-in of three ices via a single ice |
Can you update the README.md? |
@vladdoster: I've added description of |
If I understand, this would basically force users to either use of Using your example,
Say I want foo and bar targets, would it be:
or
or is build just to abstract only the following condition:
|
No, everything works correctly , it's only a pull-in ice that pulls two
other ices.
sob., 29 kwi 2023, 05:06 użytkownik vladislav doster <
***@***.***> napisał:
… @psprint <https://github.com/psprint>,
If I understand, this would not allow use of lbin, sbin, and bin-gem-node,
right?
—
Reply to this email directly, view it on GitHub
<#458 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABOE4CFVVHTFZLKHSQSKE53XDSAVDANCNFSM6AAAAAATZBSZEI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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.
Please add one test case for zunit in ices.zunit
.
@vladdoster: I've added the test for build ice. |
Description
A new shorthand for
null
/as'null'
,configure''
andmake''
ices.Motivation and Context
Providing over and over again
as'null' configure make'install'
seems boring, so a new icebuild''
has been introduced, that enables the three ices.Related Issue(s)
#345 #334 #351
Usage examples
instead of:
Types of changes
Checklist: