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

Replace macro DESFIRE_TRANSCEIVE by a function #102

Closed
wants to merge 0 commits into from

Conversation

broth-itk
Copy link
Contributor

This is to address TODO n° 3

The DESFIRE_TRANSCEIVE macro was replaced by a proper function.
Extensive tests have been made to make sure the code works like before.

@broth-itk
Copy link
Contributor Author

I think I messed up my branches - sorry! Looks like that request contains both the new function and the timeout improvement.

@smortex
Copy link
Contributor

smortex commented Jan 25, 2019

Yeah, better stick to a Pull Request per change 😉

What you can do:

# Create a new branch at the same point of history:
% git checkout -b make-timeout-configurable

% git rebase -i master
# Your editor will pop-up and list the 2 commits in your branch, remove the line which is not related to the timeout (because we are in the timeout branch)
% git push -u broth-itk make-timeout-configurable
# (assuming your remote is "broth-itk"… it may be "origin")

# You can then open a new Pull-Request with the commit in the
# make-timeout-configurable branch.

# Then, we can rewrite the history of the branch of the Pull Request:
% git checkout fix_desfire_transceive
% git rebase -i master
# Same, but now you remove the other commit
% git push -f

# The -f is required because we changed history.  This will update the pull-request.

This should put us on track :-)

@smortex
Copy link
Contributor

smortex commented Jan 25, 2019

D'oh, I did not see that you already did #101 for the other problem.

You only need to do the last 2 things:

% git rebase -i master
% git push -f

@broth-itk
Copy link
Contributor Author

Something is wrong. I think it's too late today to learn about the git magic :)

$ git push origin HEAD:fix_desfire_transceive
Username for 'https://github.com': broth-itk
Password for 'https://[email protected]': 
To https://github.com/broth-itk/libfreefare
 ! [rejected]        HEAD -> fix_desfire_transceive (non-fast-forward)
error: failed to push some refs to 'https://github.com/broth-itk/libfreefare'
hint: Updates were rejected because a pushed branch tip is behind its remote
hint: counterpart. Check out this branch and integrate the remote changes
hint: (e.g. 'git pull ...') before pushing again.
hint: See the 'Note about fast-forwards' in 'git push --help' for details.

@smortex
Copy link
Contributor

smortex commented Jan 26, 2019

I guess you are just missing a -f to force the operation. By default, git does not allow you to change something that was pushed to the outside… unless you force it to do so.

$ git push -f origin HEAD:fix_desfire_transceive

should do the trick

@broth-itk broth-itk closed this Jan 26, 2019
@broth-itk broth-itk deleted the fix_desfire_transceive branch January 26, 2019 21:39
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.

None yet

2 participants