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 OpenOCD rules copy command in App.vue #1456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rakshitx1
Copy link

@rakshitx1 rakshitx1 commented Mar 8, 2025

Description

During the ESP-IDF extension setup in VSCode on a Linux system, the user is prompted to copy the 60-openocd.rules file at the end of the configuration process. However, the generated command contains an incorrect cp option (--update=none), which results in an error.

This PR corrects the command by replacing --update=none with the valid -u option, ensuring the file copies correctly to /etc/udev/rules.d/.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Steps to test this pull request

  1. Configure the ESP-IDF extension in VSCode on a Linux system.
  2. At the end of the setup process, observe the generated command for copying 60-openocd.rules.
  3. Copy and execute the command in a terminal.

Expected behavior:

  • The command should execute successfully without errors.
  • The 60-openocd.rules file should be copied to /etc/udev/rules.d/.

Expected output:

$ sudo cp -u /path/to/60-openocd.rules /etc/udev/rules.d/

No error messages should appear, and the file should be correctly placed in the target directory.

How has this been tested?

The fix has been tested manually by:

  • Checking the updated command syntax.
  • Running the corrected command in a Linux terminal to verify it executes correctly.

Test Configuration:

  • ESP-IDF Version: 1.9.1
  • OS: Linux

Dependent components impacted by this PR

None.

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation (Not required for this fix)
  • Added Unit Test (Not required for this fix)
  • Verified on all platforms - Windows, Linux, and macOS (Only Linux is applicable for this change)

Copy link

Pull request has been marked as stale since there are no activities, and this will be closed in 5 days if there are no further activities

@github-actions github-actions bot added the stale Stale PR or Issue label Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Stale PR or Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant