Skip to content
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.

Sign up for GitHub

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

Jump to bottom

macOS builds with travis #930

Open
wants to merge 79 commits into
base: master
Choose a base branch
from
Open

macOS builds with travis #930

wants to merge 79 commits into from

Conversation

stephengroat
Copy link
Contributor

@stephengroat stephengroat commented Jan 11, 2019 •
edited
Loading

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
Copy link
Contributor Author

needs to run on appveyor to check for windows issues

@stephengroat stephengroat changed the title Macos macOS builds with travis Jan 11, 2019
This was referenced Jan 14, 2019
Merged
Merged
@stephengroat
Copy link
Contributor Author

@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?

blackb0x101 and others added 24 commits January 14, 2019 22:13
@blackb0x101
Fix linking issue on MacOs
81d1050 
Set LC_ID_DYLIB to full path to library dir under python's site-packages
@blackb0x101
Do not run exec if under MacOS
ddbd0db
@serpilliere @stephengroat
Simplifications: fix bad simplifications
4172d71
@serpilliere @stephengroat
Simplifications: simplify by default high level op
d949906 
Simplifify by default high level operators (<u, ...) with pure cst as arguments
@serpilliere @stephengroat
Support high level operators in z3 translator
f9f2a10
@serpilliere @stephengroat
Simplifications: fix hl cond reduction
bc5e45d 
Match code to comment
@serpilliere @stephengroat
Test: add high level regression tests
2f64633
@serpilliere @stephengroat
Simplification: add high evel simp
b04082d
@serpilliere @stephengroat
Bin_stream: add get_u8/u16/u32/u64
f3bf2e4
@serpilliere @stephengroat
Sandbox: change use_seh
bb4ec81 
Option name change: use-windows-structs instead of use-seh.
@serpilliere @stephengroat
Expression: replace arg by ptr in ExprMem
324ee16
@serpilliere @stephengroat
Jitter/Vm: add get/set u8/u16/u32/u64
ca3a5c3
@serpilliere @stephengroat
All: updt api
5439fe6
@GAJaloyan @stephengroat
adding immediate postdominators computation.
afa80dd
@serpilliere @stephengroat
ExprMem: fix remaining .arg accesses
09ce49a
@w4kfu @stephengroat
fix TypeError in C translator
1860c62
@w4kfu @stephengroat
add zeroextent if destination size is 32
a422fd6
@serpilliere @stephengroat
Simple: add multiplication simplification
ba275da
@commial @stephengroat
Release v0.1.0
304db31
@w4kfu @stephengroat
change return type of cntleadzeros to uint64_t
e05a46e
@gitter-badger @stephengroat
Add Gitter badge
1c37ee8
@w4kfu @stephengroat
fix simplification that delete affectation
d5aa9e9 
Revert "X86 sem : mov assignexpr rot / shift"
@w4kfu @stephengroat
fix TypeError in C translator
abab0b8
@w4kfu @stephengroat
X86 SEM ROL/SHL/SHLD/... add assignation
d80b846
stephengroat and others added 21 commits January 14, 2019 13:32
@stephengroat
use reference
e7a5974 
https://packaging.python.org/guides/supporting-windows-using-appveyor/#appveyor-yml
@stephengroat
appveyor cleanup
99d5d5a
@stephengroat
remove unnecessary apt packages from travis
dedc9fb
@dnutiu @stephengroat
Fixing miasm build for macOS
227b4d6
@serpilliere @stephengroat
MSP430: Explicit flags
974d8ad
@serpilliere @stephengroat
Armt: fix cbnz
74e48cf
@serpilliere @stephengroat
Armt: fix itt exprloc
628393a
@serpilliere @stephengroat
ARM: add mnemonics
3ca07f0
@commial @stephengroat
Disasm/full: ircfg_a and ssa.graph are the same object, use only one …
42dc0e9 
…denomination
@serpilliere @stephengroat
README: update appveyor cea-sec
7ce1ad2
@nofiv @stephengroat
Update arch.py
0712afb 
Instruction prefix wasn't shown when to_string method was used since instruction_x86 used just the inherited one(https://github.com/cea-sec/miasm/blob/master/miasm2/arch/x86/arch.py#L452) from instruction(https://github.com/cea-sec/miasm/blob/master/miasm2/core/cpu.py#L997) which doesn't support these prefixes.
@stephengroat
Add osx to travis
9bf6450
@stephengroat
fix indent
00b6700
@stephengroat
indent fix
90c182d
@stephengroat
parallel tests
ae37dd5 
should improve test speed
@serpilliere @stephengroat
Jitter: fix pc update in trace mode
a616567
@serpilliere @stephengroat
Core: rm deprecated module
72ccc16
@commial @stephengroat
Test_all: add a TAG for python jitter based tests
5e7acf5
@commial @stephengroat
Travis: avoid redundancy between parallel tests cases
01db99f
@stephengroat
test all in osx
2bf18e2
@stephengroat
remove separate build cmd from appveyor
b283668 
install should trigger
@commial
Copy link
Contributor

commial commented Jan 16, 2019

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

@XVilka
Copy link
Contributor

XVilka commented Mar 5, 2020

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

@serpilliere
Copy link
Contributor

@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 ?

@XVilka
Copy link
Contributor

XVilka commented Mar 5, 2020

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

@serpilliere
Copy link
Contributor

No problem!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.