miasm icon indicating copy to clipboard operation
miasm copied to clipboard

macOS builds with travis

Open stephengroat opened this issue 6 years ago • 7 comments

closes #438, closes #875, closes #912

still an issue with gcc tests on macos

https://travis-ci.org/cea-sec/miasm/jobs/478366262#L928

stephengroat avatar Jan 11 '19 14:01 stephengroat

needs to run on appveyor to check for windows issues

stephengroat avatar Jan 11 '19 14:01 stephengroat

@commial this works, but the gcc test cases seem to be failing (see https://travis-ci.org/cea-sec/miasm/jobs/479469186#L925)

do those have to be working for osx? if not, could you make a MIASM_TEST_EXTRA_ARG variable for https://github.com/cea-sec/miasm/pull/930/files#diff-354f30a63fb0907d4ad57269548329e3R33 so that we could limit to osx job to non gcc cases and get everything passing?

stephengroat avatar Jan 14 '19 16:01 stephengroat

Not sure to understand what you're asking. If you want to try without gcc, you just have to remove: -o qemu -t python,llvm -o gcc -t qemu,long

And replace -o long by -o long -t gcc. -o is whitelisting, -t is blacklisting.

Regarding the necessity of gcc on MacOS X, I'm not sure. If we have at least one of the fast jitter (either CC based or LLVM), it should be OK. Thoughts @serpilliere ?

Note: maybe it could work with Clang or specific options. For instance, on Windows, it is actually using cl with others flags. Options can be customized in JitCore_Gcc.add_block

commial avatar Jan 16 '19 14:01 commial

  • Should be rebased I think
  • Travis now supports Windows too, so you might extend with it too.

XVilka avatar Mar 05 '20 06:03 XVilka

@XVilka : note that we already have windows regressions tests run on appveyor. Do you propose removing appveyor in favor of travis? Can you argu on this ?

serpilliere avatar Mar 05 '20 07:03 serpilliere

No, I am not insisting. Missed the AppVeyor thing, sorry.

XVilka avatar Mar 05 '20 07:03 XVilka

No problem!

serpilliere avatar Mar 05 '20 09:03 serpilliere