Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Add Vagrantfile for ubuntu artful 32-bit #2211

Merged
merged 1 commit into from
Aug 24, 2018

Conversation

mpranj
Copy link
Member

@mpranj mpranj commented Aug 24, 2018

Vagrantfile for debugging #1973.

Build image inside cripts/vagrant/ubuntu/artful32/:

vagrant up
vagrant ssh

Steps to reproduce errors:

mkdir /vagrant/build
cd /vagrant/build

export INSTALL_DIR="$PWD/install"
export SYSTEM_DIR="$PWD/kdbsystem"
export PATH=$PATH:"$PWD/bin"
export LD_LIBRARY_PATH="$PWD/lib"
export GTEST_ROOT="/opt/gtest"

cmake -DENABLE_COVERAGE=OFF -DENABLE_OPTIMIZATIONS=OFF -DENABLE_DEBUG=ON -DENABLE_LOGGER=ON -DBUILD_STATIC=OFF -DCMAKE_INSTALL_PREFIX="$INSTALL_DIR" -DKDB_DB_SYSTEM="$SYSTEM_DIR" -DINSTALL_SYSTEM_FILES=OFF -DPLUGINS="ALL" -DTOOLS="ALL" ..

make -j4
make run_all

@markus2330
Copy link
Contributor

Thank you for the PR! It is always good to have more reproducibility.

Can you please place everything below scripts/vagrant? e.g. scripts/vagrant/artful32 is okay.

@mpranj
Copy link
Member Author

mpranj commented Aug 24, 2018

@markus2330 sure. It was never intended to be merged. If you want to merge it, I will place it nicely. I just did not want to move the other Vagrantfile around.

@markus2330
Copy link
Contributor

Yes, if the file is used by someone I think it would be useful to have it in the repo. It is not that big and its easier if files are in master and not somewhere hidden in PRs.

In docker we also named the subdirectories with distribution/codename. Maybe we create the same structure for Vagrant?

@mpranj mpranj changed the title WIP: add Vagrantfile for ubuntu artful 32-bit Add Vagrantfile for ubuntu artful 32-bit Aug 24, 2018
@markus2330
Copy link
Contributor

Please add something to release notes 😄

Btw. the same structure would be debian/jessie64/Vagrantfile and ubuntu/artful32/Vagrantfile

@mpranj mpranj force-pushed the artful32 branch 2 times, most recently from f520887 to 51fc721 Compare August 24, 2018 17:35
@mpranj
Copy link
Member Author

mpranj commented Aug 24, 2018

I was a bit too hasty there. It should be ready now.

@markus2330 markus2330 merged commit 50f4c66 into ElektraInitiative:master Aug 24, 2018
@markus2330
Copy link
Contributor

Thank you!

@mpranj mpranj deleted the artful32 branch April 21, 2021 12:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants