Page MenuHomeSolus

Update gd to 2.3.0
ClosedPublic

Authored by YakoYakoYokuYoku on Jul 7 2020, 3:57 AM.
Tags
None
Referenced Files
F11018995: D9196.id22282.diff
Sun, Aug 6, 4:39 PM
F11017662: D9196.id22152.diff
Sun, Aug 6, 4:39 AM
F10943885: D9196.id22282.diff
Fri, Jul 14, 9:28 PM
F10942157: D9196.id.diff
Fri, Jul 14, 9:44 AM
F10916381: D9196.id22300.diff
Jul 8 2023, 11:49 AM
F10909784: D9196.id22282.diff
Jul 7 2023, 3:02 AM
F10909733: D9196.id.diff
Jul 7 2023, 3:00 AM
F10874122: D9196.id22300.diff
Jun 19 2023, 10:49 PM
Subscribers

Details

Summary

Changed the build system to CMake and Ninja.
Link to Changelog here.

Signed-off-by: Martin Reboredo <yakoyoku@gmail.com>

Test Plan

Rebuilt libgphoto against it and run these commands

pngtogd in.png out.gd && gdtopng out.gd eg1.png && eog eg1.png
pngtogd2 in.png out.gd2 1024 1 && gd2topng out.gd2 eg2.png && eog eg2.png

with full success.

Diff Detail

Repository
R1697 gd
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

YakoYakoYokuYoku updated this revision to Diff 22153.
YakoYakoYokuYoku edited the test plan for this revision. (Show Details)

Corrected Markdown oopsie.

JoshStrobl added inline comments.
package.yml
33

This should not be necessary unless they're not respecting DCMAKE_LIB_SUFFIX.

I'll submit patches to their GitHub repo so the next version will have a better CMake build.

package.yml
33

DCMAKE_LIB_SUFFIX wasn't respected so this should stay as it is.

package.yml
25

Without looking at their CMakeLists.txt file, it isn't immediately clear what this does. Can we replace this with a patch?

Otherwise LGTM. Thank you for the patch instead of the sed, makes it much easier to figure out the intent :)

pspec_x86_64.xml
36

INSTALL, README* can be nuked.

This revision now requires changes to proceed.Jul 14 2020, 10:48 AM
This revision is now accepted and ready to land.Jul 15 2020, 10:51 AM
This revision was automatically updated to reflect the committed changes.