all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#51480: lib(png|gif|jpeg)-version not defined
@ 2021-10-29 13:54 Stefan Kangas
  2021-10-29 14:12 ` Eli Zaretskii
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Kangas @ 2021-10-29 13:54 UTC (permalink / raw)
  To: 51480

Severity: minor

I'm not sure if this is a bug, or just me who is missing something
obvious.

In image.c, I see the following:

      DEFSYM (Qlibpng_version, "libpng-version");
      Fset (Qlibpng_version,
    #if HAVE_PNG
            make_fixnum (PNG_LIBPNG_VER)
    #else
            make_fixnum (-1)
    #endif
            );
      DEFSYM (Qlibgif_version, "libgif-version");
      Fset (Qlibgif_version,
    #ifdef HAVE_GIF
            make_fixnum (GIFLIB_MAJOR * 10000
                         + GIFLIB_MINOR * 100
                         + GIFLIB_RELEASE)
    #else
            make_fixnum (-1)
    #endif
            );
      DEFSYM (Qlibjpeg_version, "libjpeg-version");
      Fset (Qlibjpeg_version,
    #if HAVE_JPEG
            make_fixnum (JPEG_LIB_VERSION)
    #else
            make_fixnum (-1)
    #endif
            );
    #endif

But AFAICT, this doesn't really do anything?  I can't find those
variables, the only one I have is `libgnutls-version'.  So should those
definitions be DEFVAR's, or am I missing something?


In GNU Emacs 29.0.50 (build 2, x86_64-pc-linux-gnu, GTK+ Version
3.24.24, cairo version 1.16.0)
 of 2021-10-29 built on joffe
Repository revision: 0aa9f478e962e3e2d146871fa37267709404c52e
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: Debian GNU/Linux 11 (bullseye)

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY
INOTIFY PDUMPER PNG RSVG SECCOMP SOUND THREADS TIFF TOOLKIT_SCROLL_BARS
WEBP X11 XDBE XIM XPM GTK3 ZLIB

Important settings:
  value of $LC_COLLATE: C
  value of $LC_CTYPE: sv_SE.UTF-8
  value of $LC_TIME: C
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#51480: lib(png|gif|jpeg)-version not defined
  2021-10-29 13:54 bug#51480: lib(png|gif|jpeg)-version not defined Stefan Kangas
@ 2021-10-29 14:12 ` Eli Zaretskii
  2021-10-29 14:33   ` Stefan Kangas
  0 siblings, 1 reply; 3+ messages in thread
From: Eli Zaretskii @ 2021-10-29 14:12 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 51480

> From: Stefan Kangas <stefan@marxist.se>
> Date: Fri, 29 Oct 2021 06:54:59 -0700
> 
> In image.c, I see the following:
> 
>       DEFSYM (Qlibpng_version, "libpng-version");
>       Fset (Qlibpng_version,
>     #if HAVE_PNG
>             make_fixnum (PNG_LIBPNG_VER)
>     #else
>             make_fixnum (-1)
>     #endif
>             );
>       DEFSYM (Qlibgif_version, "libgif-version");
>       Fset (Qlibgif_version,
>     #ifdef HAVE_GIF
>             make_fixnum (GIFLIB_MAJOR * 10000
>                          + GIFLIB_MINOR * 100
>                          + GIFLIB_RELEASE)
>     #else
>             make_fixnum (-1)
>     #endif
>             );
>       DEFSYM (Qlibjpeg_version, "libjpeg-version");
>       Fset (Qlibjpeg_version,
>     #if HAVE_JPEG
>             make_fixnum (JPEG_LIB_VERSION)
>     #else
>             make_fixnum (-1)
>     #endif
>             );
>     #endif
> 
> But AFAICT, this doesn't really do anything?  I can't find those
> variables, the only one I have is `libgnutls-version'.  So should those
> definitions be DEFVAR's, or am I missing something?

They are only defined on MS-Windows, because they are needed for
loading the correct DLL.  See w32-win.el.





^ permalink raw reply	[flat|nested] 3+ messages in thread

* bug#51480: lib(png|gif|jpeg)-version not defined
  2021-10-29 14:12 ` Eli Zaretskii
@ 2021-10-29 14:33   ` Stefan Kangas
  0 siblings, 0 replies; 3+ messages in thread
From: Stefan Kangas @ 2021-10-29 14:33 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 51480

tags 51480
close 51480
thanks

Eli Zaretskii <eliz@gnu.org> writes:

> They are only defined on MS-Windows, because they are needed for
> loading the correct DLL.  See w32-win.el.

Ugh, I'm blind.

Sorry for the noise, closing.





^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-29 14:33 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-29 13:54 bug#51480: lib(png|gif|jpeg)-version not defined Stefan Kangas
2021-10-29 14:12 ` Eli Zaretskii
2021-10-29 14:33   ` Stefan Kangas

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.