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.

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

Allow building with newer versions of clang/llvm tools #6

Merged
merged 1 commit into from
Sep 21, 2021

Conversation

ragusaa
Copy link
Contributor

@ragusaa ragusaa commented Sep 1, 2021

Freebsd packages naming convention use the format clang80, instead of
clang-8, so check for that, as well as allowing newer versions of clang.
Also, do not require clang libraries for building the compiler.

@lgtm-com
Copy link

lgtm-com bot commented Sep 1, 2021

This pull request introduces 2 alerts when merging 7fe92ec into 215e591 - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'

@ragusaa ragusaa force-pushed the CLAM-1548-bcc-freebsd branch from 7fe92ec to 92b181b Compare September 2, 2021 15:55
@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 2 alerts when merging 92b181b into 215e591 - view on LGTM.com

new alerts:

  • 2 for Except block handles 'BaseException'

@ragusaa ragusaa force-pushed the CLAM-1548-bcc-freebsd branch from 250dc9e to 25975e2 Compare September 2, 2021 21:30
@val-ms val-ms self-requested a review September 2, 2021 21:35
Copy link
Contributor

@val-ms val-ms left a comment

Choose a reason for hiding this comment

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

I haven't tried using it with this PR yet. I can do that next, or wait for you to update it.

clambcc/clambc-compiler.py Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
clambcc/clambc-compiler.py Outdated Show resolved Hide resolved
Freebsd packages naming convention use the format clang80, instead of
clang-8, so check for that, as well as allowing newer versions of clang.
Also, do not require clang libraries for building the compiler.
@ragusaa ragusaa force-pushed the CLAM-1548-bcc-freebsd branch from d233347 to 9243c16 Compare September 3, 2021 18:46
@ragusaa ragusaa merged commit 4b3280b into Cisco-Talos:main Sep 21, 2021
@ragusaa ragusaa deleted the CLAM-1548-bcc-freebsd branch September 21, 2021 17:54
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