On 10/03/2022 14:58, Karel Gardas wrote:
On 3/10/22 14:52, Sebastian Huber wrote:
On 10/03/2022 14:49, Karel Gardas wrote:
On 3/10/22 14:45, Sebastian Huber wrote:
On 10/03/2022 14:41, Karel Gardas wrote:
On 3/10/22 13:42, Sebastian Huber wrote:
On 10/03/2022 13:18, Karel Gardas wrote:
Hello Sebastian,
https://github.com/kgardas/rtems/tree/stm32h7-hal-update --
contains all required/recommended by you. Please recheck and
merge if possible.
Is there a line ending change in the STM code update patches?
Which line ending have the new files?
New files preserve line endings of original projects which were
source for HAL code. That is LF.
The original files were from an archive and not a Git repository.
Should they be converted first to UNIX line endings before they are
updated?
Hold on! You don't need to do anything besides the git merge. All
files needed are there. I already did the HAL code update! All files
are as provided in original projects besides few changes and besides
reapplied patches from the past. At least that was whole point of the
work...
The problem is that for example
https://github.com/kgardas/rtems/commit/d45f034fc379bcb8fecdecc1cb44bd851d6c4949
changes the line endings from DOS to UNIX AND performs the update.
This should be two commits. First the change to UNIX line endings and
then the update.
Yes, but the problem was that original files were too chaotic to do
anything about it -- in fact HAL files were using mixed line endings.
So, the only way out of the chaos was to:
1) replace all HAL files with new files
Yes, this is fine, but if all the new files have now UNIX line endings
why didn't you change the existing files to UNIX line endings before the
update? I guess you copied the files from a Git repository. Can't you
first change the existing files to UNIX line endings and make a commit.
Then copy the files from your upstream Git repository.
2) reapply patches done by RTEMS community over HAL files
I did exactly this. So now, you have new HAL + patches on top of that.
This is fine except that the update commits do two steps in one.
--
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.hu...@embedded-brains.de
phone: +49-89-18 94 741 - 16
fax: +49-89-18 94 741 - 08
Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel
