libretro / blastem

Upstream tracking repo of BlastEm, the fast and accurate Genesis emulator, with libretro specific changes
https://www.retrodev.com/blastem/
GNU General Public License v3.0
53 stars 25 forks source link

[osx] fix build failure for osx due to sed command for %.db.c rule #31

Closed fuzzard closed 3 years ago

fuzzard commented 3 years ago

OSX doesn't like having the file input for sed before the command options. Fails with error:

sed: -e: No such file or directory
sed: s/"/\\"/g: No such file or directory
sed: -e: No such file or directory
sed: s/^\(.*\)$/"\1\\n"/: No such file or directory
sed: -e1s/^\(.*\)$/const char rom_db_data[] = \1/: No such file or directory
sed: -e: No such file or directory
sed: $s/^\(.*\)$/\1;/: No such file or directory

Solution is to move the $< input to the end.

Have build tested using Kodi's libretro system. I'm not 100% sure whether this will have any adverse effects on other platforms that libretro builds, so any feedback around that would be good. I dont want to introduce a failure upstream for systems i cant test against.

inactive123 commented 3 years ago

Thanks for this!