New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JasPer uchar definition is gone #41
Comments
What version of JasPer are you using? Unless my eyes are playing tricks on me, there is no uchar on the line to which you are referring in the most recent version of JasPer on the master branch. As best I can tell, this issue must have been fixed a while ago. Incidentally, I could not help but notice your comment saying that you are updating your code to work with MSVC++ 2015. I don't know if this would be of any help to you but I am trying to put together a MSBuild project file for JasPer so that it can be built with the AppVeyor CI platform in order to allow some automated testing for MSVC environments (which I cannot do on my own machine since I don't have MSVC on it). The problem that I am having is that I have no idea how to setup the appropriate project file(s) since I am using an autotools build. If you have any words of wisdom in this regard, they would be most welcome. |
just to confirm, I see the same issue when I compile against the latest jasPer release (1.900.28).
The
|
OK, should now be sorted out in my cmake_cleanup branch. I'm intending that this stuff gets merged into the main repo, but until then feel free to cherrypick the commit: 8b5e3cb |
@mdadams wrote:
JasPer 1.900.26, as per this Gentoo version bump.
This is DevIL source code, not JasPer code. @bcampbell wrote in 8b5e3cb:
That's always a good solution. 😄 Thanks for the quick fix! I'll inform Gentoo. |
@bcampbell and @gagern: If Ubuntu is a concern for you, then the best thing to do would be to put pressure on Ubuntu to upgrade from the very obsolete 1.900.1 version of JasPer (which is about 10 years old) to the latest version, which is currently 1.900.29. I submitted a bug request to Ubuntu a while ago asking them to upgrade the version of JasPer being used: |
Thanks, @mdadams - I think the issue is further upstream, in Debian. I'm pretty sure that ubuntu is just relying on the debian packaging. The JasPer package in debian does seem to have a maintainer, but is still on 1.900.1. A bug report there might help move things along... I suspect it'd carry more weight coming from you, but I'm happy to submit one if you'd prefer. |
@gagern: keep in mind that my cmake_cleanup branch isn't (yet) merged into the official DevIL master branch, so it's probably not quite ready for distros to have at it ;-) |
@bcampbell I would appreciate very much if you could submit a bug report in Debian to request an upgrade to the latest version of JasPer (which is currently 1.900.29). You can point out that 1.900.1 is about 10 years old and the 1.900.29 (which is much more recent) has many bug fixes and some new features that are missing from 1.900.1. |
In response to jasper-software/jasper#72, JasPer no longer defines
uchar
but instead definesjas_uchar
. Quoting @mdadams from there:Current DevIL won't compile against latest JasPer due to things like this cast.
The text was updated successfully, but these errors were encountered: