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

Fix: IEP-1014 Unable to create a serial port object from the invalid port #808

Merged
merged 2 commits into from
Aug 11, 2023

Conversation

kolipakakondal
Copy link
Collaborator

@kolipakakondal kolipakakondal commented Aug 1, 2023

Description

Fix the below error. This happens when you're trying to open an serial port which doesn't exist. /dev/cu.usbserial-00001414B does not exist in my serial port list, but this was my last saved serial port in the wizard. When I try to launch the serial launch target wizard, it will throw the following error.

!STACK 0
com.fazecast.jSerialComm.SerialPortInvalidPortException: Unable to create a serial port object from the invalid port descriptor: /dev/cu.usbserial-00001414B
	at com.fazecast.jSerialComm.SerialPort.getCommPort(SerialPort.java:410)
	at com.espressif.idf.launch.serial.ui.internal.NewSerialFlashTargetWizardPage.createControl(NewSerialFlashTargetWizardPage.java:158)
	at org.eclipse.jface.wizard.Wizard.createPageControls(Wizard.java:178)
	at org.eclipse.jface.wizard.WizardDialog.createPageControls(WizardDialog.java:744)
	at org.eclipse.jface.wizard.WizardDialog.createContents(WizardDialog.java:636)

Fixes # (IEP-1014)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How has this been tested?

  • Choose some board esp32 c3
  • Select launch target, build and flash the project
  • Choose some other board. For example: esp32 wrover kit
  • Now try to launch the serial launch target wizard by clicking the edit button. It won't launch the wizard since the port doesn't exist.

Test Configuration:

  • ESP-IDF Version: 5.0
  • OS (Windows,Linux and macOS): macOS

Dependent components impacted by this PR:

  • Component 1
  • Component 2

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

@kolipakakondal kolipakakondal merged commit 838f4f8 into master Aug 11, 2023
3 of 4 checks passed
@kolipakakondal kolipakakondal deleted the IEP-1014 branch May 10, 2024 15:50
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.

1 participant