@Bruno Since last autumn (~September) main line vigra should support
c++17, but any distribution release is of course likely to not have
included that yet. But I guess it will fix itself in Fedora some time.
(wrt msvc, I was referring to what Thomas wrote.)
--
You received this bug notification
I'm not using msvc. This error is with fedora, but it looks like vigra
in Ubuntu has already been patched to support c++17. The fedora vigra
maintainer needs to copy some Ubuntu patches.
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to
Thank you for merging and testing!
The part that you omitted wasn't very important, but I am surprised that it
causes issues with msvc (I had not tested that).
I am not surprised, but slightly relieved that not only my but also your test
cases pass!
--
You received this bug notification
Thanks for testing.
I also closed the mentioned tickets.
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Title:
sometimes there are black patches/corners in blended images
Status in
It also passes the test case I uploaded here:
https://bugs.launchpad.net/enblend/+bug/785803/comments/8
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Title:
sometimes there are
Can confirm that the patched enblend passes all the tests that I
uploaded here:
https://bugs.launchpad.net/enblend/+bug/721136/comments/18
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Yes, I replaced the `throw(PreconditionViolation)` with
`noexcept(false)` in both vigra locations and now enblend builds.
I'm using enblend cmake, as the autoconf stuff needed bootstrapping and I
couldn't remember how to do it.
Haven't tested the bugfix yet, I seem to remember I created test
@Bruno: Are you building with automake or with CMake? (On Ubuntu with
CMake it works. But it seems they are providing a patched vigra library,
where this offending line is missing.)
@Lukas: I applied now most of your patches and pushed the changes to the
repository. Thank you again for the
Yes, that is weird. You're quoting vigra code,
...
#if _MSC_VER >= 1900 || __cplusplus >= 201103L
noexcept(false)
#else
throw(PreconditionViolation)
#endif
...
and given that enblend uses c++17, the offending line shouldn't be used.
Does that problem not occur without my
I can't get the latest mercurial with patches to build, this doesn't
really make sense to me as enblend requires c++17:
```
/usr/include/vigra/separableconvolution.hxx:1413:13: error: ISO C++17 does not
allow dynamic exception specifications
1413 | throw(PreconditionViolation)
p3
** Patch added: "patch3.diff"
https://bugs.launchpad.net/enblend/+bug/2053287/+attachment/5746516/+files/patch3.diff
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Title:
p2
** Patch added: "patch2.diff"
https://bugs.launchpad.net/enblend/+bug/2053287/+attachment/5746515/+files/patch2.diff
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Title:
p4
** Patch added: "patch4.diff"
https://bugs.launchpad.net/enblend/+bug/2053287/+attachment/5746517/+files/patch4.diff
--
You received this bug notification because you are a member of Hugin Bug
Hunters, which is subscribed to Enblend.
https://bugs.launchpad.net/bugs/2053287
Title:
13 matches
Mail list logo