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

Feature 203 #211

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Feature 203 #211

wants to merge 10 commits into from

Conversation

Kampi
Copy link
Collaborator

@Kampi Kampi commented Dec 27, 2023

No description provided.

- Remove detailed instructions from GETTING_STARTED because they are located in the Wiki now
- Add POSIX build to init script
- Add Git-Graph plugin to VS Code configuration
- Add dependencies for POSIX GUI
- Add nRF Command Line Tools
- Add neccessary ports for debugger

Signed-off-by: Daniel Kampert <DanielKampert@kampis-elektroecke.de>
- Remove detailed instructions from GETTING_STARTED because they are located in the Wiki now
- Add POSIX build to init script
- Add Git-Graph plugin to VS Code configuration
- Add dependencies for POSIX GUI
- Add nRF Command Line Tools
- Replace static path in default build configurations with dynamic path
- Add .bashrc for container initialization
- Add Doxygen Documentation Generator plugin
- Add remote GDB interface

Signed-off-by: Daniel Kampert <DanielKampert@kampis-elektroecke.de>
- Add rev1, rev2 and rev3 build
- Bind rev3 and rev4 debug configuration to debug launch in nRF Connect plugin
- Add options to enable / disable specific initialization stepts
- Remove old settings.json because it´s handled by the dev container

Signed-off-by: Daniel Kampert <DanielKampert@kampis-elektroecke.de>
@Kampi Kampi requested a review from jakkra December 27, 2023 18:05
Copy link
Owner

@jakkra jakkra left a comment

Choose a reason for hiding this comment

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

Changing IP not being picked up
image
image

.vscode/settings.json Show resolved Hide resolved
- [Getting Started](#getting-started)
- [Setting up the environment](#setting-up-the-environment)
- [Using the development container (recommended)](#using-the-development-container-recommended)
- [Without using the development container](#without-using-the-development-container)
Copy link
Owner

Choose a reason for hiding this comment

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

Before merging Wiki needs to fully updated according to the removed instructions here.

.devcontainer/README.md Outdated Show resolved Hide resolved
.devcontainer/README.md Show resolved Hide resolved
.devcontainer/init.sh Outdated Show resolved Hide resolved
.devcontainer/.bashrc Outdated Show resolved Hide resolved
@Kampi
Copy link
Collaborator Author

Kampi commented Dec 28, 2023

@jakkra did you change the IP address before building the container? Note that the environment will be copied once the container is built. You can retrigger it by opening the command palette and run Recreate development container without cache

- Add command line call for GDB server to README
- Add files from development container to gitignore
- Change tmp path in init script to avoid conflicts
- Replace "/workspaces/ZSWatch" path in init script with "${PWD}"

Signed-off-by: Daniel Kampert <DanielKampert@kampis-elektroecke.de>
@jakkra
Copy link
Owner

jakkra commented Dec 28, 2023

@jakkra did you change the IP address before building the container? Note that the environment will be copied once the container is built. You can retrigger it by opening the command palette and run Recreate development container without cache

I did not, will try with rebuilding it.

@Kampi
Copy link
Collaborator Author

Kampi commented Dec 28, 2023

@jakkra did you change the IP address before building the container? Note that the environment will be copied once the container is built. You can retrigger it by opening the command palette and run Recreate development container without cache

I did not, will try with rebuilding it.

It´s not a perfect solution IMO because this issue can happen. But I haven´t found a better one and AFAIK you have to set the IP address once because it doesn´t change

@Kampi Kampi marked this pull request as draft December 28, 2023 13:07
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.

None yet

2 participants