freertos_apps icon indicating copy to clipboard operation
freertos_apps copied to clipboard

Porting stm32f4_disco on dashing

Open RaKalel opened this issue 4 years ago • 10 comments

As per my conversation with @pablogs9.

RaKalel avatar Aug 07 '20 07:08 RaKalel

Thank you very much for this contribution, @RaKalel. May I please ask you to document the included 3rd-party source-code in 3rd-party-licenses.txt as part of this pull request?

ralph-lange avatar Aug 07 '20 07:08 ralph-lange

Hi @ralph-lange , so you want me to add the the third part source code like in the foxy branch to my branch (stm32 code licence) ? Once again I am sorry for these questions I am quite new to this.

RaKalel avatar Aug 07 '20 08:08 RaKalel

I was not aware that the branch of this PR was created before 3rd-party-licenses.txt has been introduced. I'm sorry! This will lead to a conflict. In this case it is probably the best if you undo the last commit (git reset --soft HEAD~1), delete the 3rd-party-licenses.txt, and force re-push the branch without this commit (git push --force ...). Then, we merge your PR and kindly ask you to provide the details on the included third-party software in a second PR, please. I'll restructure the 3rd-party-licenses.txt file in the meanwhile (cf. PR https://github.com/micro-ROS/freertos_apps/pull/21) so that you will know exactly where to put this information.

ralph-lange avatar Aug 07 '20 11:08 ralph-lange

@RaKalel, in the second PR mentioned above, also please add your name to the NOTICE file (cf. CONTRIBUTING.md).

ralph-lange avatar Aug 07 '20 11:08 ralph-lange

I'll check and test this port after the summer holidays. And I'll integrate it here and in the build system.

pablogs9 avatar Aug 07 '20 11:08 pablogs9

I'll check and test this port after the summer holidays. And I'll integrate it here and in the build system.

@RaKalel, given this delay (@pablogs9 has really earned his vacation!), you may consider to even rebase the PR with respect to the recent changes on the default branch foxy, i.e. undo the last commit as described above, pull the foxy branch, rebase your branch, add the license information to 3rd-party-licenses.txt and add your name to the NOTICE file. See https://stackoverflow.com/a/7929473 for an more detailed explanation on the procedure.

ralph-lange avatar Aug 07 '20 11:08 ralph-lange

So I undid my last commit and removed the third party licenses.txt. So now do I create a new pull request with the foxy branch as as the base? (I am afraid that I will create a big mess for you guys )

I tryed rebasing my branch as per stackoverflow.

RaKalel avatar Aug 07 '20 12:08 RaKalel

Thank you for your reply, @RaKalel. Unfortunately, the rebasing did not work properly. The following sequence clones the original repo, than adds your fork as second origin and rebases your work on top of the latest foxy branch without the last commit. Everything is stored in a new branch, which can be either pushed as new branch or be used to overwrite the existing branch:

git clone https://github.com/micro-ROS/freertos_apps.git
cd freertos_apps/
git remote add origin-fork https://github.com/RaKalel/freertos_apps.git
git pull origin-fork 
git checkout a5c8feb
git checkout -b stm32f4_disco_new
git rebase foxy
git push -f origin-fork stm32f4_disco_new 

Can you give this sequence a try, please?

ralph-lange avatar Aug 11 '20 19:08 ralph-lange

HI @ralph-lange , thanks for your instructions. I followed them and now I have a new branch "stm32f4_disco_new". So do I need to redo a PR from the new branch or do I overwrite the old 'stm32f4_disco' ?

RaKalel avatar Aug 12 '20 11:08 RaKalel

You can overwrite the existing remote branch by git push -f origin-fork stm32f4_disco_new:stm32f4_disco, cf. https://stackoverflow.com/questions/5738797/how-can-i-push-a-local-git-branch-to-a-remote-with-a-different-name-easily

ralph-lange avatar Aug 12 '20 14:08 ralph-lange