-
Notifications
You must be signed in to change notification settings - Fork 6
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
Tools install script PoC #284
base: main
Are you sure you want to change the base?
Conversation
examples/docker/src/local.env
Outdated
@@ -1,10 +1,12 @@ | |||
# Variables shared to both AEM Compose and Task tool | |||
|
|||
AEM_AUTHOR_ACTIVE=true |
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 remove these flags added
pkg/project/common/aemw
Outdated
@@ -124,4 +124,5 @@ export MSYS2_ENV_CONV_EXCL="*" | |||
# Execute AEM Compose CLI | |||
# ======================= | |||
|
|||
aem "$@" | |||
AEM=$(whereis aem | awk '{print $2}') |
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.
In tf aemc this is run without issues. In vscode ext it also is doable to run it without amendment. Please revert this
# Function to create 'tools/AEM.xml' in IntelliJ installations | ||
create_aem_xml() { | ||
local intellij_dirs=( | ||
"$HOME/Library/Application Support/JetBrains/IntelliJIdea"*/tools |
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.
/tools
does not exist by default; please create it and assume existence of parent dir only
No description provided.