Skip to content

Conversation

jgwilson42
Copy link

@jgwilson42 jgwilson42 commented Mar 23, 2017

The current instructions don't work for compiling on RedHat and need a small tweak to work for Ubuntu. I have tested these instructions on RedHat 7 and Ubuntu 16.10.

@jgwilson42 jgwilson42 changed the title Updates for GCC 6 Updates to compiling instructions for GCC 6 Mar 23, 2017
On Red Hat/Fedora or derivates, do

yum install gcc gcc-c++ flex bison perl-libwww-perl patch
yum install gcc gcc-c++ flex bison perl-libwww-perl patch devtoolset-6
Copy link

Choose a reason for hiding this comment

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

I think that when you use devtoolset-6 you can remove some of dependencies. YUM should be fine with

yum install flex bison perl-libwww-perl patch devtoolset-6

Copy link
Author

Choose a reason for hiding this comment

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

Removing gcc and gcc-c++ makes the regression tests fail without the devtoolset-6 enabled. I cannot immediately see if this is a problem with the tests or with the cbmc binary. @kroening please let me know if you would like me to investigate further or you are happy with the instructions as is.

cd cbmc-git/src
make minisat2-download
make CXX=g++-6
scl enable devtoolset-6 bash
Copy link

Choose a reason for hiding this comment

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

and then you probably need to run this before first make

Copy link
Author

Choose a reason for hiding this comment

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

agreed this is required, if the above requested change is made.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, let's go for the "scl enable devtoolset-6 bash" change; it's better that the regressions work out-of-the box

Copy link
Author

Choose a reason for hiding this comment

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

In which case I am happy with the state of this PR. The version here will ensure that the regression tests run under most scenarios.

Please let me know if you need anything else.

@kroening kroening merged commit 7e406cd into diffblue:master Mar 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants