all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Re: [Emacs-diffs] master 5d4c539: Add lcms2 interface
       [not found]   ` <jwvmv5y38nt.fsf-monnier+emacsdiffs@gnu.org>
@ 2017-09-13 16:01     ` Eli Zaretskii
  2017-09-13 16:23       ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2017-09-13 16:01 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: mvoteiza, emacs-devel

> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Date: Wed, 13 Sep 2017 11:46:46 -0400
> Cc: Mark Oteiza <mvoteiza@udel.edu>
> 
> >     configure.ac: Add boilerplate for configuring and detecting liblcms2.
> 
> Can't this be a module?

Only if we don't want this functionality in Emacs where the library is
available.

> The whole point of the addition of the module interface was so that we
> don't end up with the situation where installing Emacs either requires
> a zillion libraries that are "hard-coded" (tho dynamically linked) into
> the binary, or it only requires a few libraries but then the other
> features can't be used.

To me, modules are for users to extend Emacs, not for us to add
functionality.  It makes very little sense to have anything that is in
core via modules.  I see only disadvantages there: the code is much
less readable, and has more limitations on what it can do.



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

* Re: [Emacs-diffs] master 5d4c539: Add lcms2 interface
  2017-09-13 16:01     ` [Emacs-diffs] master 5d4c539: Add lcms2 interface Eli Zaretskii
@ 2017-09-13 16:23       ` Stefan Monnier
  2017-09-13 17:07         ` Paul Eggert
  2017-09-13 19:24         ` Eli Zaretskii
  0 siblings, 2 replies; 5+ messages in thread
From: Stefan Monnier @ 2017-09-13 16:23 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: mvoteiza, emacs-devel

>> >     configure.ac: Add boilerplate for configuring and detecting liblcms2.
>> Can't this be a module?
> Only if we don't want this functionality in Emacs where the library is
> available.

Hmm... indeed I now see that liblcms2 is already linked into Emacs if we
use the Gtk GUI.  It's not linked if you use the Lucid GUI, OTOH.

> To me, modules are for users to extend Emacs, not for us to add
> functionality.

That's one of the benefits.  The other is that you don't need to choose
at compile-time which libraries will be (and have to be) available at
run-time.

In Windows, we already do dynamic-loading of libraries such as gnutls,
so the issue is less serious, but in GNU/Linux it's always a hassle.

> It makes very little sense to have anything that is in
> core via modules.  I see only disadvantages there: the code is much
> less readable, and has more limitations on what it can do.

That calls for improving the module system, not circumventing it.


        Stefan



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

* Re: [Emacs-diffs] master 5d4c539: Add lcms2 interface
  2017-09-13 16:23       ` Stefan Monnier
@ 2017-09-13 17:07         ` Paul Eggert
  2017-09-13 17:44           ` Stefan Monnier
  2017-09-13 19:24         ` Eli Zaretskii
  1 sibling, 1 reply; 5+ messages in thread
From: Paul Eggert @ 2017-09-13 17:07 UTC (permalink / raw)
  To: Stefan Monnier, Eli Zaretskii; +Cc: mvoteiza, emacs-devel

Stefan Monnier wrote:
> Hmm... indeed I now see that liblcms2 is already linked into Emacs if we
> use the Gtk GUI.  It's not linked if you use the Lucid GUI, OTOH.

Would it address your concerns if we enabled lcms2 only when Gtk is already in use?



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

* Re: [Emacs-diffs] master 5d4c539: Add lcms2 interface
  2017-09-13 17:07         ` Paul Eggert
@ 2017-09-13 17:44           ` Stefan Monnier
  0 siblings, 0 replies; 5+ messages in thread
From: Stefan Monnier @ 2017-09-13 17:44 UTC (permalink / raw)
  To: Paul Eggert; +Cc: mvoteiza, Eli Zaretskii, emacs-devel

>> Hmm... indeed I now see that liblcms2 is already linked into Emacs if we
>> use the Gtk GUI.  It's not linked if you use the Lucid GUI, OTOH.
> Would it address your concerns if we enabled lcms2 only when Gtk is
> already in use?

No, my comment was more general than specifically aimed at lmcs2.


        Stefan



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

* Re: [Emacs-diffs] master 5d4c539: Add lcms2 interface
  2017-09-13 16:23       ` Stefan Monnier
  2017-09-13 17:07         ` Paul Eggert
@ 2017-09-13 19:24         ` Eli Zaretskii
  1 sibling, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2017-09-13 19:24 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: mvoteiza, emacs-devel

> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Cc: emacs-devel@gnu.org, mvoteiza@udel.edu
> Date: Wed, 13 Sep 2017 12:23:34 -0400
> 
> > To me, modules are for users to extend Emacs, not for us to add
> > functionality.
> 
> That's one of the benefits.  The other is that you don't need to choose
> at compile-time which libraries will be (and have to be) available at
> run-time.
> 
> In Windows, we already do dynamic-loading of libraries such as gnutls,
> so the issue is less serious, but in GNU/Linux it's always a hassle.

I think we should indeed welcome patches to migrate to a similar
dynamic-loading scheme on Posix hosts.  Most of the code can be simply
lifted from the Windows port, with minimal changes.

> > It makes very little sense to have anything that is in
> > core via modules.  I see only disadvantages there: the code is much
> > less readable, and has more limitations on what it can do.
> 
> That calls for improving the module system, not circumventing it.

That would be welcome as well, but the issues were identified long
ago, and no one volunteered to work on that until now.  Moreover, I
understand some of us actually think it would be wrong to go that way.



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

end of thread, other threads:[~2017-09-13 19:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20170913143143.30953.94864@vcs0.savannah.gnu.org>
     [not found] ` <20170913143145.6483C2068F@vcs0.savannah.gnu.org>
     [not found]   ` <jwvmv5y38nt.fsf-monnier+emacsdiffs@gnu.org>
2017-09-13 16:01     ` [Emacs-diffs] master 5d4c539: Add lcms2 interface Eli Zaretskii
2017-09-13 16:23       ` Stefan Monnier
2017-09-13 17:07         ` Paul Eggert
2017-09-13 17:44           ` Stefan Monnier
2017-09-13 19:24         ` Eli Zaretskii

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.