8000 feature: Detect board port change after upload by cmaglie · Pull Request #2253 · arduino/arduino-cli · GitHub
[go: up one dir, main page]
Skip to content

feature: Detect board port change after upload #2253

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

Merged
merged 18 commits into from
Aug 18, 2023
Merged

feature: Detect board port change after upload #2253

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
d7e9242
UploadResponse now has 'oneof' clause for better API design
cmaglie Jul 31, 2023
97d9e24
Added scaffolding to return updated-port after upload
cmaglie Jul 31, 2023
bfb2fb4
Upload port change detection (first draft)
cmaglie Aug 1, 2023
f7eef3b
Simplified port detection using a Future-style abstraction
cmaglie Aug 2, 2023
83dcade
Perform watcher-flush higher in the call tree
cmaglie Aug 2, 2023
3312627
Do not infer upload port if 'upload.wait_for_upload_port' is false
cmaglie Aug 2, 2023
a087862
Further simplified port detection subroutine structure
cmaglie Aug 2, 2023
9f8d266
fixed linter issue
cmaglie Aug 2, 2023
5896fba
Always return an updatedUploadPort.
cmaglie Aug 9, 2023
990d93a
Updated docs
cmaglie Aug 9, 2023
a750bdc
Perform a deep-copy of upload ports where needed.
cmaglie Aug 11, 2023
d1869b6
When updating `actualPort` address, update also the address label.
cmaglie Aug 11, 2023
af873e0
Fixed some potential nil pointer exceptions
cmaglie Aug 11, 2023
f8394fb
Further simplified board watcher
cmaglie Aug 11, 2023
e9e5fbd
Before returning from upload, check if the port is still alive
cmaglie Aug 14, 2023
69d9ae7
Apply suggestions from code review
cmaglie Aug 16, 2023
e12a068
Fixed nil exception
cmaglie Aug 16, 2023
862bbe2
Improved tracking algorithm for upload-port reconnection
cmaglie Aug 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixed linter issue
  • Loading branch information
@cmaglie
cmaglie committed Aug 9, 2023
commit 9f8d266cf3c0b57d5f6783268ef89e61d34097ad
4 changes: 2 additions & 2 deletions commands/upload/upload.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ func runProgramAction(pme *packagemanager.Explorer,
// expected port change then run the detector.
updatedUploadPort := f.NewFuture[*rpc.Port]()
if uploadProperties.GetBoolean("upload.wait_for_upload_port") && watch != nil {
go detectUploadPort(port, watch, uploadCtx, updatedUploadPort)
go detectUploadPort(uploadCtx, port, watch, updatedUploadPort)
} else {
updatedUploadPort.Send(nil)
go f.DiscardCh(watch)
Expand Down Expand Up @@ -522,7 +522,7 @@ func runProgramAction(pme *packagemanager.Explorer,
return updatedUploadPort.Await(), nil
}

func detectUploadPort(uploadPort *rpc.Port, watch <-chan *rpc.BoardListWatchResponse, uploadCtx context.Context, result f.Future[*rpc.Port]) {
func detectUploadPort(uploadCtx context.Context, uploadPort *rpc.Port, watch <-chan *rpc.BoardListWatchResponse, result f.Future[*rpc.Port]) {
log := logrus.WithField("task", "port_detection")
log.Tracef("Detecting new board port after upload")

Expand Down

Footer

© 2025 GitHub, Inc.
0