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: 14455
Joined: Wed May 27, 2009 8:48 am
Location: Salerno, Italy
Has thanked: 1076 times
Been thanked: 922 times
Contact:

Re: Plans on merging the latest FatFS updates?

Postby Giovanni » Fri Jan 13, 2017 9:59 am

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: 14455
Joined: Wed May 27, 2009 8:48 am
Location: Salerno, Italy
Has thanked: 1076 times
Been thanked: 922 times
Contact:

Re: Plans on merging the latest FatFS updates?

Postby Giovanni » Fri Jan 13, 2017 2:28 pm

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
Location: Luleå, Sweden
Has thanked: 5 times
Been thanked: 6 times
Contact:

Re: Plans on merging the latest FatFS updates?

Postby Korken » Sat Jan 28, 2017 2:32 pm

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?

Postby lerela » Tue Mar 21, 2017 7:12 pm

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 170 times

lerela
Posts: 11
Joined: Tue Mar 21, 2017 7:08 pm
Been thanked: 2 times

Re: Plans on merging the latest FatFS updates?

Postby lerela » Wed Jul 26, 2017 10:52 am

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.


Return to “Bug Reports”

Who is online

Users browsing this forum: No registered users and 20 guests