-
Notifications
You must be signed in to change notification settings - Fork 61
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
Migration to github actions #693
base: master
Are you sure you want to change the base?
Conversation
spel/scripts/virtualbox.sh
Outdated
@@ -10,7 +10,7 @@ echo "installing virtualbox guest addition dependencies" | |||
VBOX_GUEST_DEPS=(kernel-devel kernel-headers gcc perl) | |||
test "$(rpm --quiet -q bzip2)$?" -eq 0 || VBOX_GUEST_DEPS+=(bzip2) | |||
bash /tmp/retry.sh 5 yum -y install "${VBOX_GUEST_DEPS[@]}" | |||
bash /tmp/retry.sh 5 yum -y install dkms make | |||
bash /tmp/retry.sh 5 yum -y install dkms make -f Makefile.spel |
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.
This is a yum install command, not a make command, so the option isn't valid here. No change needed.
bash /tmp/retry.sh 5 yum -y install dkms make -f Makefile.spel | |
bash /tmp/retry.sh 5 yum -y install dkms make |
build/vagrant/buildspec-vagrant.yml
Outdated
@@ -12,10 +12,10 @@ env: | |||
phases: | |||
install: | |||
commands: | |||
- make -C build/vagrant install | |||
- make -C -f Makefile.spel build/vagrant install |
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.
-C
and -f
actually conflict. No need to change these, since this is actually already telling make to use the vagrant buildspec.
- make -C -f Makefile.spel build/vagrant install | |
- make -C build/vagrant install |
build/vagrant/buildspec-vagrant.yml
Outdated
build: | ||
commands: | ||
- make -C build/vagrant build | ||
- make -C -f Makefile.spel build/vagrant build |
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.
- make -C -f Makefile.spel build/vagrant build | |
- make -C build/vagrant build |
build/vagrant/build-spel-vagrant.sh
Outdated
@@ -41,7 +41,7 @@ if [[ -n "${SPEL_REPO_COMMIT:-}" ]] ; then | |||
fi | |||
|
|||
# install packer | |||
make -f Makefile.tardigrade-ci packer/install | |||
make -f Makefile.spel packer/install |
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.
make -f Makefile.spel packer/install | |
make packer/install |
build/vagrant/Makefile
Outdated
@@ -30,7 +30,7 @@ $(info SPEL_VERSION=$(SPEL_VERSION)) | |||
all: build | |||
|
|||
install: | |||
$(MAKE) -f ../../Makefile.tardigrade-ci packer/install | |||
$(MAKE) -f ../../Makefile.spel packer/install |
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.
$(MAKE) -f ../../Makefile.spel packer/install | |
$(MAKE) -f ../../Makefile packer/install |
Makefile.spel
Outdated
|
||
install: | ||
$(MAKE) -f Makefile.tardigrade-ci packer/install | ||
$(MAKE) -f Makefile.spel packer/install |
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.
$(MAKE) -f Makefile.spel packer/install | |
$(MAKE) -f Makefile packer/install |
Makefile.spel
Outdated
|
||
docs/generate: | ||
$(MAKE) -f Makefile.tardigrade-ci docs/generate | ||
$(MAKE) -f Makefile.spel docs/generate |
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.
$(MAKE) -f Makefile.spel docs/generate | |
$(MAKE) -f Makefile docs/generate |
Makefile.spel
Outdated
@@ -45,13 +45,13 @@ endif | |||
all: build | |||
|
|||
docs/lint: | |||
$(MAKE) -f Makefile.tardigrade-ci docs/lint | |||
$(MAKE) -f Makefile.spel docs/lint |
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.
$(MAKE) -f Makefile.spel docs/lint | |
$(MAKE) -f Makefile docs/lint |
No description provided.