On Wed, Feb 17, 2016 at 11:04:16AM -0800, Bryce Harrington wrote: > On Wed, Feb 17, 2016 at 12:28:48PM +0100, Emmanuel Gil Peyrot wrote: > > On Tue, Feb 16, 2016 at 10:29:34PM -0800, Bryce Harrington wrote: > > > On Tue, Feb 16, 2016 at 01:57:52AM +0000, Emmanuel Gil Peyrot wrote: > > > > It doesn’t make sense to fail the entire build when jpeglib isn’t > > > > present, so this commit makes it optional just like libwebp in the > > > > previous one, disabled with --without-jpeg and forced with --with-jpeg. > > > > > > > > Signed-off-by: Emmanuel Gil Peyrot <linkma...@linkmauve.fr> > > > > > > Is there any functional loss when jpg support is missing? > > > > Yes, Weston as a whole loses the ability to load JPEG files, be it in > > weston-image, in icons, or maybe more importantly in the background. > > > > There is no JPEG image being installed by Weston though, so only > > user-installed images would break. > > Ok, yeah that's about what you'd expect. The patch looks technically > fine to me, and presuming there's no other more fundamental lossage > involved here: > > Reviewed-by: Bryce Harrington <br...@osg.samsung.com> > > Out of curiousity does this solve a particular issue you're aware of, or > is this just for symmetry with the webp build control?
While packaging weston for some distribution, I found out that it failed to compile when jpeglib wasn’t present, and didn’t see any reason for it. WebP was actually the one I modified for symmetry, and because it’s much nicer to be able to control the dependencies which gets compiled in. :) > > Bryce -- Emmanuel Gil Peyrot _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel