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

Adds hf iclass esetblk, equivalent to hf mf esetblk #2086

Merged
merged 4 commits into from
Aug 22, 2023

Conversation

nvx
Copy link
Contributor

@nvx nvx commented Aug 21, 2023

Also some minor tweaks to make commands.json and commands.md properly on Windows. Previously on windows it would cause a difference vs on Linux due to the case of differences of the serial device.

I also noticed a similar "always makes a diff for every line" with the pm3line_vocabulary.h due to a fight between the python script adding a space at the end of each line and the rest of make style removing trailing whitespace on lines.

Also some minor tweaks to make commands.json and commands.md
properly on Windows.
@github-actions
Copy link

You are welcome to add an entry to the CHANGELOG.md as well

static int CmdHFiClassESetBlk(const char *Cmd) {
CLIParserContext *ctx;
CLIParserInit(&ctx, "hf iclass esetblk",
"Display emulator memory.\n"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe you need to update this texts to match the new command

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops missed that one!

@iceman1001
Copy link
Collaborator

Nice fixes, just need the help text to be updated then it can be merged

would actually been better with several PR's...

@nvx nvx force-pushed the feature/cmd_iclass_esetblk branch from 0586945 to bc87615 Compare August 22, 2023 16:45
@nvx
Copy link
Contributor Author

nvx commented Aug 22, 2023

would actually been better with several PR's...

yeah I thought that afterwards, wasn't intending to fix the commands build stuff but then ended up fighting it as a result of adding the new command and didn't want a thousand line diff because the tooling was doing weird stuff.

should be right to merge now that I remembered to regenerate the doc/commands.json after fixing the help text now.

@iceman1001 iceman1001 merged commit 4547a1b into RfidResearchGroup:master Aug 22, 2023
7 of 9 checks passed
@iceman1001
Copy link
Collaborator

Awesome fixes!
Small but still they were annoying ones.

@nvx nvx deleted the feature/cmd_iclass_esetblk branch August 22, 2023 20:52
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.

2 participants