CI Jenkins: activate macOS by smlng · Pull Request #6341 · RIOT-OS/RIOT

smlng · 2017-01-13T08:43:47Z

this PR activates macOS as a build target in Jenkins CI.

However there are several errors to fix first, hence this PR is currently based on #6326 and #6333, more to come ...

smlng · 2017-01-13T19:41:56Z

needs #6351, too

~~[edit]: and #6352~~

[edit2] #6364

smlng · 2017-01-18T19:53:15Z

rebased on latest master, but still depends on other PRs

smlng · 2017-01-19T08:58:35Z

put #6399 on the list of dependencies ...

miri64 · 2017-01-19T16:59:30Z

Jenkins doesn't show any results for MacOSX (unless I'm missing something)

smlng · 2017-01-20T07:57:13Z

@miri64 triggered Jenkins again, now it shows all green lights

miri64 · 2017-01-20T09:50:27Z

Click -> Ctrl+f -> "mac" -> 0 of 0. :-/

smlng · 2017-01-20T13:37:14Z

requires #6440

kYc0o · 2017-02-03T14:12:13Z

Got it, do we have a "DO NOT MERGE" label?

smlng · 2017-02-03T14:12:42Z

WIP?

[edit] that's why I set that

kYc0o · 2017-02-03T14:12:57Z

OK.

kYc0o · 2017-02-03T14:14:31Z

It needs a rebase BTW.

miri64 · 2017-02-03T14:14:50Z

Also the faild CIs give a good hint to not merge this too ;-)

kYc0o · 2017-02-03T14:17:14Z

Jenkins failed because of #4058

smlng · 2017-02-03T14:20:01Z

@kYc0o nope that was fixed in #6534, hence I rebased, but other issues might prevail, see #6473

kYc0o · 2017-02-03T14:21:23Z

Yes I know, that's why I suggested the rebase, since I know #6534 got merged meanwhile, so I just mentioned that was the reason of the failure.

smlng · 2017-02-03T14:49:50Z

@kYc0o Jenkins is happy. What about your list in #6473, close that issue?

kYc0o · 2017-02-03T14:57:58Z

The things on my list correspond rather to warnings which will be raised for other drivers that are excluded on native.

I suggest to keep the issue open until #1121 will be merged.

kYc0o · 2017-02-03T14:59:03Z

BTW, why relic works? Is thus test built on real macOS host or just clang?

smlng · 2017-02-03T15:02:37Z

@kYc0o IIRC relic is only used by tests/unittests/test_relic, but its build on bare metal macOS using clang - with success.

kYc0o · 2017-02-04T13:36:04Z

Well, in my Mac it doesn't work...

snake:unittests facosta$ make tests-relic
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Building application "tests_unittests" for "native" with MCU "native".

"/Applications/Xcode.app/Contents/Developer/usr/bin/make" -C /Users/facosta/git/RIOT2/RIOT/pkg/relic
rm -Rf /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic
mkdir -p /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic
/Users/facosta/git/RIOT2/RIOT/dist/tools/git/git-cache clone "https://github.com/relic-toolkit/relic.git" "cdcfaeef101d18c3231c3b46359c519dd72682e8" "/Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic"
Cloning into '/Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic'...
remote: Counting objects: 16213, done.
remote: Total 16213 (delta 0), reused 0 (delta 0), pack-reused 16213
Receiving objects: 100% (16213/16213), 3.69 MiB | 1.35 MiB/s, done.
Resolving deltas: 100% (13018/13018), done.
Checking connectivity... done.
HEAD is now at cdcfaee... Fix cycle counter in ARM.
touch /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic/.git-downloaded
git -C /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic checkout -f cdcfaeef101d18c3231c3b46359c519dd72682e8
HEAD is now at cdcfaee... Fix cycle counter in ARM.
git -C /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic am --ignore-whitespace "/Users/facosta/git/RIOT2/RIOT/pkg/relic"/patches/*.patch
Applying: fixed signedness of counter variable
Applying: don't redefine ALIGN
Applying: require only CMake version 2.8
touch /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic/.git-patched
./fix-util_print_wo_args.sh /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic
sed: RE error: illegal byte sequence
make[1]: *** [fix_source] Error 1
make: *** [/Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/native/relic.a] Error 2

smlng · 2017-02-05T09:58:48Z

Hi back,

Am 04.02.2017 um 14:36 schrieb Francisco Acosta ***@***.***>: Well, in my Mac it doesn't work... snake:unittests facosta$ make tests-relic Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1 Building application "tests_unittests" for "native" with MCU "native". "/Applications/Xcode.app/Contents/Developer/usr/bin/make" -C /Users/facosta/git/RIOT2/RIOT/pkg/relic rm -Rf /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic mkdir -p /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic /Users/facosta/git/RIOT2/RIOT/dist/tools/git/git-cache clone "https://github.com/relic-toolkit/relic.git" "cdcfaeef101d18c3231c3b46359c519dd72682e8" "/Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic" Cloning into '/Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic'... remote: Counting objects: 16213, done. remote: Total 16213 (delta 0), reused 0 (delta 0), pack-reused 16213 Receiving objects: 100% (16213/16213), 3.69 MiB | 1.35 MiB/s, done. Resolving deltas: 100% (13018/13018), done. Checking connectivity... done. HEAD is now at cdcfaee... Fix cycle counter in ARM. touch /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic/.git-downloaded git -C /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic checkout -f cdcfaeef101d18c3231c3b46359c519dd72682e8 HEAD is now at cdcfaee... Fix cycle counter in ARM. git -C /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic am --ignore-whitespace "/Users/facosta/git/RIOT2/RIOT/pkg/relic"/patches/*.patch Applying: fixed signedness of counter variable Applying: don't redefine ALIGN Applying: require only CMake version 2.8 touch /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic/.git-patched ./fix-util_print_wo_args.sh /Users/facosta/git/RIOT2/RIOT/tests/unittests/bin/pkg/native/relic sed: RE error: illegal byte sequence

This error typically comes from a wrong LANG setting, however, this should be fixed by `pkg/relic/os_util.sh`. I assume you are using a recent RIOT version ... Best, Sebastian [mail]: [email protected] [code]: https://github.com/smlng

kYc0o · 2017-02-06T16:45:43Z

Yes, I'm on current master and it shows that error. Have you manually configured your LANG settings? I've never touched them...

smlng · 2017-02-07T08:05:51Z

@kYc0o as said the helper script in pkg/relic/os_util.sh was added to fix params of sed command and the LANG settings. I did not tamper with that either and haven't had these problems since then ...

kYc0o · 2017-02-07T13:04:46Z

So you suggest me to run that script to remove that error or you mean that the script should run by itself while adding the pkg?

smlng · 2017-02-07T13:42:57Z

actually its the latter, so I'm a bit confused why it doesn't work for you. It does on our macOS Jenkins slave, on my MacBook and my other Mac, too. I haven't changed any language settings and sed is the macOS native one. But I use Homebrew on my machines, maybe that changes some environment settings?

kYc0o · 2017-02-07T14:14:46Z

I'd say let's merge and let me deal with my problem, since it seems it only happens on my system. Moreover, I think this is more important for Jenkins, and since he's happy, I think it's fine. So go!

kYc0o · 2017-02-07T14:15:15Z

I let @cgundogan merge it when he thinks it's OK.

miri64 · 2017-02-16T15:44:13Z

Ping @cgundogan?

smlng · 2017-02-16T15:53:35Z

I'll close this one as memo, we currently do not have sufficient resources to build every PR on macOS. However, for the planned master periodic/nightly build it should be included.

As an alternative, I'd like to get llvm toolchain supported in Jenkins and enabled for every PR see #6600.

smlng added Area: CI Area: Continuous Integration of RIOT components OS: Mac OS X Host OS: This PR/issue concerns usage of RIOT with Mac OS X as a host system State: waiting for other PR State: The PR requires another PR to be merged first labels Jan 13, 2017

smlng force-pushed the pr/ci/activate_macos branch 3 times, most recently from 45ca82f to b18c3ea Compare January 13, 2017 10:10

smlng changed the title ~~CI Jenkins: activate macos~~ CI Jenkins: activate macOS Jan 13, 2017

smlng mentioned this pull request Jan 13, 2017

pkg/tinydtls: add compiler flags to Makefile.include #6351

Merged

This was referenced Jan 13, 2017

pkg/ccn-lite: add patch for compiler error #6352

Closed

pkg: udpate ccn-lite version to include patches for RIOT #6364

Closed

CI: Automatic testing of native on different platforms #3392

Open

smlng force-pushed the pr/ci/activate_macos branch 2 times, most recently from 921f7cf to 582981d Compare January 18, 2017 19:52

smlng mentioned this pull request Jan 19, 2017

timex: unambiguous time conversion macros #6399

Merged

smlng force-pushed the pr/ci/activate_macos branch from 582981d to 0ea9a37 Compare January 19, 2017 16:37

smlng added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed State: waiting for other PR State: The PR requires another PR to be merged first labels Jan 19, 2017

smlng added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jan 20, 2017

smlng force-pushed the pr/ci/activate_macos branch from 0ea9a37 to 8d434c4 Compare January 20, 2017 12:48

smlng removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 20, 2017

smlng force-pushed the pr/ci/activate_macos branch from 8d434c4 to 0bb84fc Compare January 20, 2017 13:36

smlng mentioned this pull request Jan 20, 2017

unittests: fix heatshrink compiler error on macOS #6440

Merged

ci: activate macOS

9015020

smlng force-pushed the pr/ci/activate_macos branch from 35a1c16 to 9015020 Compare February 3, 2017 14:16

miri64 assigned cgundogan Feb 3, 2017

miri64 requested a review from cgundogan February 3, 2017 16:34

smlng closed this Feb 16, 2017

smlng added State: archived State: The PR has been archived for possible future re-adaptation and removed State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet labels Feb 16, 2017

smlng deleted the pr/ci/activate_macos branch May 31, 2017 18:03

Conversation

smlng commented Jan 13, 2017

Uh oh!

smlng commented Jan 13, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

smlng commented Jan 18, 2017

Uh oh!

smlng commented Jan 19, 2017

Uh oh!

miri64 commented Jan 19, 2017

Uh oh!

smlng commented Jan 20, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

miri64 commented Jan 20, 2017

Uh oh!

smlng commented Jan 20, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

smlng commented Feb 3, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

miri64 commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

smlng commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

smlng commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 3, 2017

Uh oh!

smlng commented Feb 3, 2017

Uh oh!

kYc0o commented Feb 4, 2017

Uh oh!

smlng commented Feb 5, 2017 via email

Uh oh!

kYc0o commented Feb 6, 2017

Uh oh!

smlng commented Feb 7, 2017

Uh oh!

kYc0o commented Feb 7, 2017

Uh oh!

smlng commented Feb 7, 2017

Uh oh!

kYc0o commented Feb 7, 2017

Uh oh!

kYc0o commented Feb 7, 2017

Uh oh!

miri64 commented Feb 16, 2017

Uh oh!

smlng commented Feb 16, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

smlng commented Jan 13, 2017 •

edited

Loading

smlng commented Jan 20, 2017 •

edited

Loading

smlng commented Feb 3, 2017 •

edited

Loading