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

cpp improvements #688

Merged
merged 7 commits into from
Nov 6, 2023
Merged

cpp improvements #688

merged 7 commits into from
Nov 6, 2023

Conversation

prabhu
Copy link
Collaborator

@prabhu prabhu commented Nov 2, 2023 •
edited
Loading

  • Update atom to bring improved performance in header analysis mode.
  • Use generic type instead of conan consistently
  • Load evinser dynamically. Fixes [deno] repotests have started failing #686
  • Support for BOM Profiles. See ADVANCED.md

Testing

git clone https://github.com/CycloneDX/cdxgen
cd cdxgen
git checkout feature/tune-cpp
npm install
node bin/cdxgen.js -t c -o bom.json <path to c repo>
node bin/cdxgen.js -t c --deep -o bom.json <path to c repo>

What to report

  • Performance difference between normal and deep mode
  • Any crashes

@prabhu prabhu mentioned this pull request Nov 2, 2023
Open
@prabhu prabhu added the help wanted Extra attention is needed label Nov 2, 2023
@prabhu
Use generic type for c/c++ instead of conan
16139fe 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

Updated wrapdb

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

Load evinser dynamically. Fixes #686

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

Update atom to bring improved peformance in header analysis mode

Support for vcpkg.json and dependency tree construction while generating cpp modules

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>

Update atom which brings the symlink fix

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Bug fix
b7889fb 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Added repo tests
87af155 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Support for BOM profile
f37bae6 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Support for BOM profile
d3e284d 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Try again with jenkins tests
14894f3 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu
Use options instead of args
c0b7ff0 
Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@prabhu prabhu merged commit 1f7794b into master Nov 6, 2023
23 checks passed
@prabhu prabhu deleted the feature/tune-cpp branch November 6, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cerrussell cerrussell Awaiting requested review from cerrussell

Assignees
No one assigned
Labels
help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[deno] repotests have started failing
1 participant
@prabhu