Rockbox

is there any reason why we limit this path to 32 chars except to save some memory?

nope, but then there is another more serious problem which wasnt obvious when I filed this… the Playlists directory should be in the user writable area so it should be ~/.config/rockbox.org/Playlists by default.

I think it actually is, it’s supposed to be anyways.

“/usr/local/share/rockbox/xxx” is translated into $HOME/.config/rockbox.org/xxx (if it exists) at even though it says /usr/local/share.

The path doesn’t mean much anyway. Only the the basename of the file (without path and extensions) is actually stored in RAM. From there on it looks in $HOME/.config/rockbox.org/xxx, then /usr/local/share/rockbox for the files/folders.

not entirely working version. wpsbuild.pl needs to be fixed so the cabbie theme uses the correct path prefix. I’m not sure the autoconf.h change is correct also.

   filepathchanges.diff (5.9 KiB)

What does that patch? I thought the bug this task is about was fixed.

config.cfg storing real paths which it shouldnt. it should actually store them in the same way regular targets do (i.e wps: /.rockbox/wps/cabbiev2.wps) so you could take a .cfg from your dap and use it on raaa. right now its using /<Rb>/ but the prefix is irrelevant. the important thing is it shouldnt be storing /usr/local/….

As I said in my previous post, it doesn’t actually use the path from the .cfg. It actually ignores the path (and extension) and only looks at the basefile name in the usual search path order.

that isnt 100% true as I found out. that full path is currently used if the get_user_file_dir() isnt called correctly (like forgetting IS_FILE). and anyway that is more reason to not put the full path in the config.

cleaner version which make ROCKBOX_DIR point back to the same as normal builds, ROCKBOX_SHARE_PATH is the old ROCKBOX_DIR. doing it this way means no scripts need to be fixed to work, and configs can be shared without any problem.

I find it a bit annoying that you really need 2 MAX_PATH buffers in the caller to make it work though.

   filepathchanges.diff (7.7 KiB)

This is more of a proof of concept than committable patch… remove all get_user_file_path() calls in apps/ and transparently change the filename in open() and opendir() to get the right path. sdl app only just now

   filepathchanges.diff (34.6 KiB)

This changes the android path to /data/data/org.rockbox/.rockbox amd transparently changes file paths called with open() and diropen().
tested with android and sdl.

make reconf doesnt work though and that needs to be fixed before commiting

   filepathchanges.diff (43 KiB)

this version fixes make reconf. please test

   filepathchanges.diff (44.2 KiB)

sync

   filepathchanges.diff (43 KiB)

Sync and extended to wrap all IO functions that take a path.

Works very well, I think I’m going to commit in a day or two.

   app-paths.patch (49.5 KiB)