8000 Create vscode debug environment by CzBalti · Pull Request #208 · eclipse/cloe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Create vscode debug environment #208

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
Merged
merged 1 commit into from
Jun 12, 2024
Merged

Create vscode debug environment #208

merged 1 commit into from
Jun 12, 2024

Conversation

CzBalti
Copy link
Contributor
@CzBalti CzBalti commented Dec 15, 2023 •
edited
Loading
  • conan build_type should be "Debug"
  • integrate cloe speedup tips
  • make configure should be executed to get proper hinting
    • when running the first time or after a make clean it should be make configure CONAN_OPTIONS="-c tools.build:skip_test=1" to disable the tests. (The build time is faster without running the tests)
  • create VERSION file?

@CzBalti
Copy link
Contributor Author
CzBalti commented Dec 15, 2023

Sometimes breakpoints are jumping a few lines up when debugging, could changing the build_type to Debug in conan profile improve it?

@cassava
Copy link
Contributor
cassava commented Jan 8, 2024

Sometimes breakpoints are jumping a few lines up when debugging, could changing the build_type to Debug in conan profile improve it?

Yes.

cassava
cassava reviewed Jan 8, 2024
View reviewed changes
cassava
cassava reviewed Jan 8, 2024
View reviewed changes
.vscode/launch.json Outdated
8000
"name": "Debug cloe",
"type": "cppdbg",
"request": "launch",
"program": "${workspaceFolder}/build/RelWithDebInfo/bin/cloe-engine",
Copy link
Contributor

Choose a reason for hiding this comment

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

RelWithDebInfo is a variable we can't control.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would use CLOE_ENGINE env variable to get the path to the used cloe-engine executable

cassava
cassava reviewed Jan 26, 2024
View reviewed changes
cassava
cassava reviewed Jan 26, 2024
View reviewed changes
cassava
cassava reviewed Jan 26, 2024
View reviewed changes
cassava
cassava reviewed Jan 26, 2024
View reviewed changes
@cassava cassava force-pushed the ben/develop branch 3 times, most recently from abcf824 to 1e02a5c Compare March 28, 2024 15:27
@cassava cassava force-pushed the ben/develop branch 2 times, most recently from 523cffa to 25c8196 Compare April 22, 2024 11:03
@cassava
Copy link
Contributor
cassava commented Apr 22, 2024

I rebased on ben/develop, so you'll have to reset to your origin.

@cassava cassava force-pushed the ben/develop branch 3 times, most recently from 6c2435d to a544f9f Compare May 15, 2024 07:02
@cassava cassava force-pushed the ben/develop branch 4 times, most recently from e7e0ea4 to d672d71 Compare May 16, 2024 22:09
@CzBalti CzBalti force-pushed the dani/vscode branch 3 times, most recently from 89c2745 to bfc2f6c Compare May 23, 2024 12:07
cassava
cassava approved these changes Jun 3, 2024
View reviewed changes
Copy link
Contributor
@cassava cassava left a comment

Choose a reason for hiding this comment

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

Ok, lets go with this for now. I still don't like the missing final newline though. That will mess with other tools. Can you fix that for these too?

@cassava cassava merged commit bb129dd into eclipse:ben/develop Jun 12, 2024
6 checks passed
@cassava cassava added this to the 0.25.0 milestone Jun 12, 2024
@cassava cassava self-assigned this Jun 12, 2024
@cassava cassava added the enhancement New feature or request label Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cassava cassava cassava approved these changes

@tobifalk tobifalk Awaiting requested review from tobifalk

Assignees

@cassava cassava

Labels
enhancement New feature or request
Projects
None yet
Milestone
0.25.0
Development

Successfully merging this pull request may close these issues.
2 participants
@CzBalti @cassava
0