Closed isZumpo closed 3 months ago
The recent updates enhance the development environment by configuring port forwarding, updating user permissions, and refining scripts for package installations and server startup. The Dockerfile now supports a dev-user
and birdnet-user
with appropriate permissions. Additionally, the Makefile
and scripts have been updated to streamline development server operations, including audio device selection.
File/Path | Change Summary |
---|---|
.devcontainer/devcontainer.json |
Added port forwarding for port 8080, updated CGO_CFLAGS path, set container user to dev-user , enabled remote user UID update, and modified post-attach command. |
.devcontainer/postCreateCommand.sh |
Updated package installations, added new dependencies (alsa-utils , dialog ), and adjusted installation commands. |
.devcontainer/start_dev_server.sh |
Enhanced to capture and list audio recording devices, prompt user selection, and start the development server with the chosen device. |
Dockerfile |
Created new user dev-user , adjusted permissions and directories, switched to birdnet-user for running the application, updated working directory, and managed ownership in COPY commands. |
Makefile |
Modified dev_server target to include REALTIME_ARGS before invoking air realtime . |
No sequence diagrams generated as the changes are too varied and simple.
In the land of code, where changes bloom,
A new user joins the Docker room.
Ports now open, scripts anew,
Withalsa-utils
anddialog
too.
The server hums, devices sing,
Development soars on Rabbit's wing.
🐰✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Will have to try out the last commit a bit before opening this pull request up for merge/review. I want to be sure that there will be no file permission issues when mounting in volumes from the host
Will have to try out the last commit a bit before opening this pull request up for merge/review. I want to be sure that there will be no file permission issues when mounting in volumes from the host
Ok skipping that last commit since it requires more verification. All else should be good though :)