remove bundled git sources #104
Conversation
|
Optional Future improvement: switching from global link_libraries() / include_directories() to target_link_libraries(osl PRIVATE ...) would avoid leaking flags to consumers. Only a nice-to-have, not a blocker. |
| lib/libpspmath/src/vfpu_quaternion_sample_hermite.c | ||
| lib/libpspmath/src/vfpu_quaternion_hermite_tangent.c | ||
| ) | ||
| set(GIFLIB_SOURCES |
There was a problem hiding this comment.
Good call removing the vendored giflib. Just one concern, if we switch to external giflib, we must ensure CMake fails explicitly when giflib is not found, rather than silently disabling GIF support. Suggest find_package(GIF REQUIRED) when GIF is part of OSL_IMAGE_FORMAT_LOADERS.
| list(APPEND OSL_IMAGE_FORMATS_SUPPORTED ${IMAGE_FORMAT}) | ||
| if(${IMAGE_FORMAT} IN_LIST OSL_IMAGE_FORMAT_LOADERS) | ||
| add_compile_definitions(-DOSL_IMAGE_LOADER_${IMAGE_FORMAT}) | ||
| include_directories(${${IMAGE_FORMAT}_INCLUDE_DIRS}) |
There was a problem hiding this comment.
Nice move dynamically wiring loaders/writers per format. But GIF is missing in the wiring — I see PNG/JPEG handled, but no conditional block for giflib (e.g. ${GIF_INCLUDE_DIRS}, ${GIF_LIBRARIES}). If OSL_IMAGE_FORMAT_LOADERS contains GIF, we should add both add_compile_definitions(...) and target_link_libraries(... GIF::GIF or ${GIF_LIBRARIES}). Otherwise we risk unintentional loss of GIF support.
| PSP_FW_VERSION=371 | ||
|
|
||
| CFLAGS = -O2 -g -G0 -Wall | ||
| DEFINES := \ |
There was a problem hiding this comment.
All samples now explicitly link PNG/JPEG (-lpng -ljpeg -lz) 👍.
However GIF is missing (-DOSL_IMAGE_LOADER_GIF and -lgif). From a user perspective this looks like GIF support got dropped. I recommend either:
adding GIF defines + -lgif to the relevant samples, or providing at least one dedicated GIF example to prove the loader is still supported.
nonstringattribute-DOSL_IMAGE_LOADER_[FORMAT]flags-DOSL_IMAGE_WRITER_[FORMAT]flags