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.

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

Jump to bottom

EDS name mismatch on CF_ResetCountersCmd #469

Closed
2 tasks done
jphickey opened this issue Jan 31, 2025 · 0 comments · Fixed by #470
Closed
2 tasks done

EDS name mismatch on CF_ResetCountersCmd #469

jphickey opened this issue Jan 31, 2025 · 0 comments · Fixed by #470
Assignees
@jphickey
Labels
bug

Comments

@jphickey
Copy link
Contributor

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I performed a cursory search to see if the bug report is relevant, not redundant, nor in conflict with other tickets.

Describe the bug
The EDS file no longer matches the name of the CF_ResetCountersCmd. It appears a change was merged back in October (#387) that changed the name of the command from CF_ResetCmd to CF_ResetCountersCmd but the EDS file (and associated refs) were not updated as part of that, resulting in a name mismatch.

To Reproduce
Build CF using EDS, build breaks.

Expected behavior
Build should succeed

System observed on:
Debian

Reporter Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Jan 31, 2025
@jphickey jphickey added the bug label Jan 31, 2025
@jphickey jphickey mentioned this issue Jan 31, 2025
Merged
2 tasks
@dzbaker dzbaker closed this as completed in fcf1089 Feb 6, 2025
dzbaker added a commit that referenced this issue Feb 6, 2025
@dzbaker
Merge pull request #470 from jphickey/fix-469-resetcounters
eb9a8ef 
Fix #469, rename CF_ResetCmd to CF_ResetCountersCmd in EDS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #469, rename CF_ResetCmd to CF_ResetCountersCmd in EDS jphickey/CF
1 participant
@jphickey