Bug#690942: python-pgmagick: Image.attribute does not set the attribute but instead appends to it

[Juraj Komacka]

This should be fixed in upstream with current stable release of
GraphicsMagick 1.3.26

2017-04-05 19:57 GMT+02:00 Juraj Komačka :

> According to http://www.graphicsmagick.org/api/attribute.html#setimageatt
> ribute special handling for EXIF:Orientation has been introduced in
> http://hg.code.sf.net/p/graphicsmagick/code/rev/90be78bb60fb
>
> > The new EXIF:Orientation attribute replaces the existing value rather
> than being concatenated to it as when setting other attributes.
>
> I have reproduced exactly same behaviour for GM core C API with current
> stable from upstream, the problem is resolved in latest developments
> snapshot from (1.4-020170315). AFAIK no additional change is needed in
> python-pgmagick or libgraphicsmagick++.
>
> --
> To unsubscribe, send mail to 690942-unsubscr...@bugs.debian.org.
>

Bug#690942: python-pgmagick: Image.attribute does not set the attribute but instead appends to it

[Juraj Komačka]

According to 
http://www.graphicsmagick.org/api/attribute.html#setimageattribute 
special handling for EXIF:Orientation has been introduced in 
http://hg.code.sf.net/p/graphicsmagick/code/rev/90be78bb60fb


> The new EXIF:Orientation attribute replaces the existing value rather 
than being concatenated to it as when setting other attributes.


I have reproduced exactly same behaviour for GM core C API with current 
stable from upstream, the problem is resolved in latest developments 
snapshot from (1.4-020170315). AFAIK no additional change is needed in 
python-pgmagick or libgraphicsmagick++.

Bug#690942: python-pgmagick: Image.attribute does not set the attribute but instead appends to it

[Juraj Komačka]

Hi all,

this is beahviour described in GM's docs - see 
http://www.graphicsmagick.org/Magick++/Image.html#attribute


Retested with libgraphicsmagick++1 (=1.3.25-8) and python-pgmagick 
(=0.6.4-1) from Stretch.


> If the named attribute already exists, the provided text is appended 
to the existing attribute text.


In this unreleased commit - 
http://hg.code.sf.net/p/graphicsmagick/code/rev/c9067f61966c was 
implemented functionality required to remove image attributes
> Pass NULL to remove an existing text attribute, or to restart the 
text attribute from scratch.
so hopefully will be present in next GM release (after 1.3.25) - all 
this applies to the C++ library.


After that maintainer of python-pgmagick should be made aware of this 
new functionality (by filling an issue at 
https://github.com/hhatto/pgmagick) to make this functionality available 
in Python bindings.

Bug#690942: python-pgmagick: Image.attribute does not set the attribute but instead appends to it

[W. Martin Borgert]

Hi Michael,

could you please retest with 0.6.2-1 from unstable?
And add a "found" or "notfound" to the bug report?

TIA!

Bug#690942: python-pgmagick: Image.attribute does not set the attribute but instead appends to it

[Michael Fladischer]

Package: python-pgmagick
Version: 0.5.1-1+b1
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Dear Maintainer,

I'm trying to change an EXIF value by doing this:

In [1]: from pgmagick import Image

In [2]: img = Image('2_topright_x.jpg')

In [3]: img.attribute('EXIF:Orientation')
Out[3]: '2'

In [4]: img.attribute('EXIF:Orientation', '1')

In [5]: img.attribute('EXIF:Orientation')
Out[5]: '21'

As you can seen the '1' was attached to the original value instead of 
overwriting it, resulting in an invalid EXIF tag.

- -- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.5-trunk-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages python-pgmagick depends on:
ii  libboost-python1.49.0  1.49.0-3.1
ii  libc6  2.13-35
ii  libgcc11:4.7.2-4
ii  libgraphicsmagick++3   1.3.16-1.1
ii  libstdc++6 4.7.2-4
ii  python 2.7.3-2
ii  python2.6  2.6.8-0.2
ii  python2.7  2.7.3-5

python-pgmagick recommends no packages.

python-pgmagick suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iEYEARECAAYFAlCBGY8ACgkQeJ3z1zFMUGYkRACeN4DDtZ4g8LZo6AqodNN/Czqz
RlUAnjfSaEJ/syvqIw601uJjyoLYSrY8
=Rufy
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org