Skip to content

travis: switch to Bionic #1075

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

Merged
merged 6 commits into from
Jul 1, 2020
Merged

travis: switch to Bionic #1075

merged 6 commits into from
Jul 1, 2020

Conversation

evverx
Copy link
Member

@evverx evverx commented Jun 28, 2020

To judge from fc412fe, back in 2016 it was the latest version
of Ubuntu available on Travis CI. Now build tools there are too old
for testing purposes.

Other than that, Ubuntu Trusty hasn't been supported at https://apt.llvm.org/
since August 2019, which makes it kind of hard at this point to bring the latest clang
along with ASan, UBsan, Msan and libFuzzer to Travis CI.

evverx added 2 commits June 28, 2020 19:59
To judge from fc412fe, back in 2016 it was the latest version
of Ubuntu available on Travis CI. Now build tools there are too old
for testing purposes.

Other than that, Ubuntu Trusty hasn't been supported at https://apt.llvm.org/
since August 2019, which makes it kind of hard at this point to bring the latest clang
along with ASan, UBsan, Msan and libFuzzer to Travis CI.
@evverx
Copy link
Member Author

evverx commented Jun 28, 2020

Blocked by #1076 and #1077

@evverx evverx force-pushed the bionic branch 3 times, most recently from 8f9eb0f to 3a695fa Compare June 29, 2020 14:11
to get around an ASan bug mentioned in util-linux#1076
to prepare the infrastructure for util-linux#1068
@evverx
Copy link
Member Author

evverx commented Jun 29, 2020

It seems to be working now. Tomorrow hopefully I'll replace the hard-coded values to make it easier to switch to the next version of clang/gcc once they're available.

@karelzak
Copy link
Collaborator

OK, I'll wait for the update.

by just changing the compiler option in .travis.yml

In https://travis-ci.org/github/karelzak/util-linux/builds/703664282 I switched
to gcc-9/clang-9 by simply applying the following patch:
```
diff --git a/.travis.yml b/.travis.yml
index ec12847..12a247d 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -7,8 +7,8 @@ git:
   depth: 1500

 compiler:
-  - gcc-10
-  - clang-10
+  - gcc-9
+  - clang-9

 env:
   - MAKE_CHECK="nonroot"
```
@evverx evverx force-pushed the bionic branch 4 times, most recently from 1e7f76a to e0967c5 Compare June 30, 2020 20:25
@evverx
Copy link
Member Author

evverx commented Jun 30, 2020

To judge from https://travis-ci.org/github/karelzak/util-linux/builds/703664282 where I switched from gcc-10/clang-10 to gcc-9/clang-9 by changing just the compiler option in .travis.yml

diff --git a/.travis.yml b/.travis.yml
index ec1284717..12a247d91 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -7,8 +7,8 @@ git:
   depth: 1500

 compiler:
-  - gcc-10
-  - clang-10
+  - gcc-9
+  - clang-9

 env:
   - MAKE_CHECK="nonroot"

I think it's safe to say that that part is more or less maintainable and the PR should be good to go. @karelzak could you take a look?

@evverx
Copy link
Member Author

evverx commented Jun 30, 2020

By the way, trying to make sure I didn't break anything I looked for places where travis_install is used and came across the following paragraph
https://github.com/karelzak/util-linux/blob/f9a4f93584fe6e0a8eef413c0a72b2ef7f075789/Documentation/howto-tests.txt#L129-L146

Is drone.io still used? If not, I can delete that in a follow-up PR so as not to confuse contributors.

@karelzak karelzak merged commit 96f1144 into util-linux:master Jul 1, 2020
@karelzak
Copy link
Collaborator

karelzak commented Jul 1, 2020

Merged, thanks! I have updated the Documentation/howto-tests.txt, drone.io seems dead thing -- I have added LGTM.com.

@evverx evverx deleted the bionic branch July 1, 2020 13:08
evverx added a commit to evverx/util-linux that referenced this pull request Jul 1, 2020
Backtraces like
```
misc-utils/cal.c:295:4: runtime error: signed integer overflow: 2147483647 + 5 cannot be represented in type 'int'
==1405==WARNING: invalid path to external symbolizer!
==1405==WARNING: Failed to use and restart external symbolizer!
    #0 0x4c4c09  (/home/travis/build/karelzak/util-linux/cal+0x4c4c09)
    #1 0x7f4363046b96  (/lib/x86_64-linux-gnu/libc.so.6+0x21b96)
    #2 0x41bab9  (/home/travis/build/karelzak/util-linux/cal+0x41bab9)
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior misc-utils/cal.c:295:4 in
```
aren't very helpful

It's follow-up to util-linux#1075
evverx added a commit to evverx/util-linux that referenced this pull request Jul 1, 2020
Backtraces like
```
misc-utils/cal.c:295:4: runtime error: signed integer overflow: 2147483647 + 5 cannot be represented in type 'int'
==1405==WARNING: invalid path to external symbolizer!
==1405==WARNING: Failed to use and restart external symbolizer!
    #0 0x4c4c09  (/home/travis/build/karelzak/util-linux/cal+0x4c4c09)
    #1 0x7f4363046b96  (/lib/x86_64-linux-gnu/libc.so.6+0x21b96)
    #2 0x41bab9  (/home/travis/build/karelzak/util-linux/cal+0x41bab9)
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior misc-utils/cal.c:295:4 in
```
aren't very helpful

It's follow-up to util-linux#1075
evverx added a commit to evverx/util-linux that referenced this pull request Jul 1, 2020
Backtraces like
```
misc-utils/cal.c:295:4: runtime error: signed integer overflow: 2147483647 + 5 cannot be represented in type 'int'
==1405==WARNING: invalid path to external symbolizer!
==1405==WARNING: Failed to use and restart external symbolizer!
    #0 0x4c4c09  (/home/travis/build/karelzak/util-linux/cal+0x4c4c09)
    #1 0x7f4363046b96  (/lib/x86_64-linux-gnu/libc.so.6+0x21b96)
    #2 0x41bab9  (/home/travis/build/karelzak/util-linux/cal+0x41bab9)
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior misc-utils/cal.c:295:4 in
```
aren't very helpful

It's follow-up to util-linux#1075
@karelzak
Copy link
Collaborator

karelzak commented Jul 2, 2020

Ah, yesterday I get an email to re-activate drone.io ;-) It seems it works, but all the previous settings probably lost. I'll ignore it for now.

karelzak pushed a commit that referenced this pull request Jul 3, 2020
Backtraces like
```
misc-utils/cal.c:295:4: runtime error: signed integer overflow: 2147483647 + 5 cannot be represented in type 'int'
==1405==WARNING: invalid path to external symbolizer!
==1405==WARNING: Failed to use and restart external symbolizer!
    #0 0x4c4c09  (/home/travis/build/karelzak/util-linux/cal+0x4c4c09)
    #1 0x7f4363046b96  (/lib/x86_64-linux-gnu/libc.so.6+0x21b96)
    #2 0x41bab9  (/home/travis/build/karelzak/util-linux/cal+0x41bab9)
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior misc-utils/cal.c:295:4 in
```
aren't very helpful

It's follow-up to #1075
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.

2 participants