Re: [libvirt PATCH 08/31] docs: drop %.png: %.fig rule
On Thu, Jul 09, 2020 at 05:13:47PM +0200, Michal Privoznik wrote: > On 7/9/20 4:59 PM, Daniel P. Berrangé wrote: > > On Thu, Jul 02, 2020 at 02:25:06PM +0200, Pavel Hrdina wrote: > > > convert bin is part of ImageMagick package and uses uniconvertor to > > > create png file from fig files. > > > > > > Unfortunately uniconvertor is python2 only and not available in most > > > recent distributions which makes the convert command fail with: > > > > > > sh: uniconvertor: command not found > > > /usr/bin/mv: cannot stat '/tmp/magick-1397138DRT8Pzx4Qmoc.svg': No such > > > file or directory > > > convert: delegate failed `'uniconvertor' '%i' '%o.svg'; /usr/bin/mv > > > '%o.svg' '%o'' @ error/delegate.c/InvokeDelegate/1958. > > > convert: unable to open file `/tmp/magick-1397138S8ARueJXLXkc': No such > > > file or directory @ error/constitute.c/ReadImage/605. > > > convert: no images defined `docs/migration-managed-direct.png' @ > > > error/convert.c/ConvertImageCommand/3226. > > > > > > It looks like that there are plans to somehow port uniconvertor into > > > python3 but as part of different project color-picker but the job is > > > far from complete. > > > > FYI i would support replacing the .fig files in git with .svgs > > which will avoid the extra conversion step. The .fig stuff is just > > too anachronistic to be worth worrying about. > > Actually, I think that is the right way to go instead of just removing this > rule. Because if we ever change a *.fig file we have to remember to > regenerate the corresponding .png file. And then we can drop .png files from > our repos. In practice that turned out to not be a good idea. When I did the new logos a few years back I tried having the pngs auto-generated at build time. The Inkscape SVG -> bitmap rendering was not very good quality wrt its anti-aliasing. Rendering them in GIMP gave far better results. I still think we should convert the fig to svg though, despite not using them in the build procss. Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|
Re: [libvirt PATCH 08/31] docs: drop %.png: %.fig rule
On 7/9/20 4:59 PM, Daniel P. Berrangé wrote: On Thu, Jul 02, 2020 at 02:25:06PM +0200, Pavel Hrdina wrote: convert bin is part of ImageMagick package and uses uniconvertor to create png file from fig files. Unfortunately uniconvertor is python2 only and not available in most recent distributions which makes the convert command fail with: sh: uniconvertor: command not found /usr/bin/mv: cannot stat '/tmp/magick-1397138DRT8Pzx4Qmoc.svg': No such file or directory convert: delegate failed `'uniconvertor' '%i' '%o.svg'; /usr/bin/mv '%o.svg' '%o'' @ error/delegate.c/InvokeDelegate/1958. convert: unable to open file `/tmp/magick-1397138S8ARueJXLXkc': No such file or directory @ error/constitute.c/ReadImage/605. convert: no images defined `docs/migration-managed-direct.png' @ error/convert.c/ConvertImageCommand/3226. It looks like that there are plans to somehow port uniconvertor into python3 but as part of different project color-picker but the job is far from complete. FYI i would support replacing the .fig files in git with .svgs which will avoid the extra conversion step. The .fig stuff is just too anachronistic to be worth worrying about. Actually, I think that is the right way to go instead of just removing this rule. Because if we ever change a *.fig file we have to remember to regenerate the corresponding .png file. And then we can drop .png files from our repos. Michal
Re: [libvirt PATCH 08/31] docs: drop %.png: %.fig rule
On Thu, Jul 02, 2020 at 02:25:06PM +0200, Pavel Hrdina wrote: > convert bin is part of ImageMagick package and uses uniconvertor to > create png file from fig files. > > Unfortunately uniconvertor is python2 only and not available in most > recent distributions which makes the convert command fail with: > > sh: uniconvertor: command not found > /usr/bin/mv: cannot stat '/tmp/magick-1397138DRT8Pzx4Qmoc.svg': No such file > or directory > convert: delegate failed `'uniconvertor' '%i' '%o.svg'; /usr/bin/mv '%o.svg' > '%o'' @ error/delegate.c/InvokeDelegate/1958. > convert: unable to open file `/tmp/magick-1397138S8ARueJXLXkc': No such file > or directory @ error/constitute.c/ReadImage/605. > convert: no images defined `docs/migration-managed-direct.png' @ > error/convert.c/ConvertImageCommand/3226. > > It looks like that there are plans to somehow port uniconvertor into > python3 but as part of different project color-picker but the job is > far from complete. FYI i would support replacing the .fig files in git with .svgs which will avoid the extra conversion step. The .fig stuff is just too anachronistic to be worth worrying about. Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|
Re: [libvirt PATCH 08/31] docs: drop %.png: %.fig rule
On a Thursday in 2020, Pavel Hrdina wrote: convert bin is part of ImageMagick package and uses uniconvertor to create png file from fig files. Unfortunately uniconvertor is python2 only and not available in most recent distributions which makes the convert command fail with: sh: uniconvertor: command not found /usr/bin/mv: cannot stat '/tmp/magick-1397138DRT8Pzx4Qmoc.svg': No such file or directory convert: delegate failed `'uniconvertor' '%i' '%o.svg'; /usr/bin/mv '%o.svg' '%o'' @ error/delegate.c/InvokeDelegate/1958. convert: unable to open file `/tmp/magick-1397138S8ARueJXLXkc': No such file or directory @ error/constitute.c/ReadImage/605. convert: no images defined `docs/migration-managed-direct.png' @ error/convert.c/ConvertImageCommand/3226. It looks like that there are plans to somehow port uniconvertor into python3 but as part of different project color-picker but the job is far from complete. Signed-off-by: Pavel Hrdina --- docs/Makefile.am | 3 --- 1 file changed, 3 deletions(-) Reviewed-by: Ján Tomko Jano signature.asc Description: PGP signature