Skip to content
This repository has been archived by the owner on Aug 20, 2022. It is now read-only.

update to Ubuntu 18.04 #104

Merged
merged 61 commits into from
Apr 10, 2019
Merged

update to Ubuntu 18.04 #104

merged 61 commits into from
Apr 10, 2019

Conversation

KrisThielemans
Copy link
Member

No description provided.

@KrisThielemans KrisThielemans requested review from paskino and removed request for paskino December 4, 2018 22:19
@paskino
Copy link
Contributor

paskino commented Dec 15, 2018

Vagrant suggests to use bento boxes as base boxes for different linux flavours.

We could/should change to bento/ubuntu-18.04.

@paskino
Copy link
Contributor

paskino commented Dec 17, 2018

I'm trying to vagrant up but it's failing like

A customization command failed:

["storageattach", :id, "--storagectl", "IDE", "--port", "1", "--device", "0", "--type", "dvddrive", "--medium", "emptydrive"]

The following error was experienced:

#<Vagrant::Errors::VBoxManageError: There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["storageattach", "d7c7b264-e7bd-48e5-a8e2-f991b116557b", "--storagectl", "IDE", "--port", "1", "--device", "0", "--type", "dvddrive", "--medium", "emptydrive"]

Stderr: VBoxManage.exe: error: Could not find a controller named 'IDE'
>

Please fix this customization and try again.

@paskino
Copy link
Contributor

paskino commented Feb 8, 2019

That error is due to the name of the "--storagectl", "IDE" which should be IDE Controller

@paskino
Copy link
Contributor

paskino commented Mar 15, 2019

closes #94

@paskino
Copy link
Contributor

paskino commented Mar 15, 2019

probably needs SyneRBI/SIRF-SuperBuild#220

The VM gets created correctly on the ubuntu18.04 branch of the SuperBuild

@paskino paskino mentioned this pull request Mar 18, 2019
Closed
KrisThielemans
KrisThielemans commented Apr 9, 2019
View reviewed changes
Copy link
Member Author

@KrisThielemans KrisThielemans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also will need to change SIRF_VM_VERSION in `UPDATE.sh

scripts/bootstrap_ubuntu.sh Outdated
git checkout ubuntu18.04
else
cd CCPPETMR_VM
git checkout ubuntu18.04
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will need to change

scripts/bootstrap_ubuntu.sh Outdated Show resolved Hide resolved
scripts/bootstrap_ubuntu.sh Outdated
python -m pip install spyder

chown -R $SIRFUSERNAME:users $userHOME
# the ubuntu18.04 branch latest commit
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete

vagrant/Vagrantfile Outdated Show resolved Hide resolved
CHANGES.md Show resolved Hide resolved
@KrisThielemans
Copy link
Member Author

will need to update CHANGES.md

@paskino paskino merged commit 4536ce8 into master Apr 10, 2019
@paskino paskino deleted the ubuntu18.04 branch May 8, 2019 12:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@paskino paskino paskino left review comments

@bathomas bathomas Awaiting requested review from bathomas

Assignees

@paskino paskino

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KrisThielemans @paskino