-
Notifications
You must be signed in to change notification settings - Fork 37
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
Build with CLAP functional #440
Conversation
alias clap-pillar = " <absolute pth to VM> <absolute path to image> clap"
…t, in several methods and it's working!
…each builder (peer programming with Guillermo)
build command + builders subcommands, in PRTarget
@@ -175,7 +175,8 @@ ConfigurationForPillar >> attribution: aValue [ | |||
{ #category : #accessing } | |||
ConfigurationForPillar >> baseDirectory [ | |||
"generated code" | |||
^ self propertyAt: #baseDirectory ifAbsent: [(FileSystem workingDirectory / '.')] | |||
"^ self propertyAt: #baseDirectory ifAbsent: [(FileSystem workingDirectory) / '.']" |
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.
We should remove commented code :)
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.
Done
|
||
{ #category : #'command line' } | ||
ClapPillarBuildCommand class >> commandSpecification [ | ||
<commandline> |
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.
Shouldn't this be correctly indented?
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.
Done
I do not really understand the points :) S |
@LaurineDargaud can you confirm that this PR is obsolete and #443 is the good one I should look at? Can I close this one? |
Required: clap-st project by cdlm
Build as a CLAP commandline + subcommand in PRTarget created for each builder
By default: mainRoot strategy
Order to apply flags: check list then all then main
Used flags : -a for all, -m for mainRoot (maybe useless...) + help
Seen with Damien: delete methods which create strategy (buildAll, buildMainRoot...)