Plans on merging the latest FatFS updates? Topic is solved

Report here problems in any of ChibiOS components. This forum is NOT for support.
User avatar
Giovanni
Site Admin
Posts: 14891
Joined: Wed May 27, 2009 8:48 am
Has thanked: 1202 times
Been thanked: 996 times

Re: Plans on merging the latest FatFS updates?

Post by Giovanni »

Hi,

Could you remind me what has been patched in FatFS? I wish to add the newest release which is 0.12b, it seems to compile fine without patches with your modified support files.

Giovanni
User avatar
Giovanni
Site Admin
Posts: 14891
Joined: Wed May 27, 2009 8:48 am
Has thanked: 1202 times
Been thanked: 996 times

Re: Plans on merging the latest FatFS updates?

Post by Giovanni »

Never mind, found the difference, it was ffconf.h.

Change merged, thanks.

Giovanni
User avatar
Korken
Posts: 270
Joined: Wed Apr 02, 2014 4:09 pm
Has thanked: 5 times
Been thanked: 6 times

Re: Plans on merging the latest FatFS updates?

Post by Korken »

Thanks for merging!
I have been semi-away from here a while, sorry for not answering. :)
lerela
Posts: 11
Joined: Tue Mar 21, 2017 7:08 pm
Been thanked: 2 times

Re: Plans on merging the latest FatFS updates?

Post by lerela »

Hi Giovanni,

There's an issue with fatfs-0.12b-patched.7z. It includes ffconf.h in src/, which overwrites the user-defined ffconf.h.
In fatfs-0.10b-patched.7z, this file was renamed to ffconf_template.h to avoid this issue.

Attached is a version with ffconf.h renamed to ffconf_template.h.

-l
Attachments
fatfs-0.12b-patched.7z
(1.39 MiB) Downloaded 429 times
lerela
Posts: 11
Joined: Tue Mar 21, 2017 7:08 pm
Been thanked: 2 times

Re: Plans on merging the latest FatFS updates?

Post by lerela »

FYI, issue raised in previous post has not been fixed in current 17.6.x branch. FatFS archive still contains ffconf.h, which overrides/breaks user-defined configuration.
Post Reply