Skip to content

Add cmake for newer version of libjpeg-turbo#8102

Merged
adamjstewart merged 8 commits intospack:developfrom
s-sajid-ali:libjpeg_turbo_cmake
May 14, 2018
Merged

Add cmake for newer version of libjpeg-turbo#8102
adamjstewart merged 8 commits intospack:developfrom
s-sajid-ali:libjpeg_turbo_cmake

Conversation

@s-sajid-ali
Copy link
Copy Markdown
Contributor

Newer version uses of libjpeg-turbo uses cmake.

@s-sajid-ali s-sajid-ali changed the title Add cmake for newer version of libjepg-turbo Add cmake for newer version of libjpeg-turbo May 13, 2018
@when('@:1.5.3')
def install(self, spec, prefix):
configure('--prefix=%s' % prefix)
make
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be make(), it's a function, not a statement.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

library that implements JPEG image encoding, decoding and
transcoding."""
# https://github.com/libjpeg-turbo/libjpeg-turbo/blob/master/BUILDING.md
homepage = "https://github.com/libjpeg-turbo/libjpeg-turbo"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would actually use https://libjpeg-turbo.org/ as the homepage. That's what the previous homepage redirects to.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason the building instructions for the 1.6.x was broken from there but that's fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants