emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Kaushal Modi <kaushal.modi@gmail.com>
To: Adam Porter <adam@alphapapa.net>
Cc: emacs-org list <emacs-orgmode@gnu.org>
Subject: Re: bug#32722: bug#32722: bug#32722: 26.1; Org-publish depend on non-free platform ?
Date: Fri, 21 Sep 2018 17:30:27 -0400	[thread overview]
Message-ID: <CAFyQvY3ppssUrUA-O5yC4ytrQ_defd7HS41S3yPKajfAJJU+dQ@mail.gmail.com> (raw)
In-Reply-To: <87zhwbehtn.fsf@alphapapa.net>

On Thu, Sep 20, 2018 at 9:55 PM Adam Porter <adam@alphapapa.net> wrote:

> 1.  Why not?  I just git-blamed this line in ox-html.el:
>
>   (declare-function htmlize-region "ext:htmlize" (beg end))
>
> It's from February, 2012.  That's 6 and a half years, at least, that
> that code has been present.  Why are we suddenly concerned about it?

I just so happens that someone recently filed this bug report:
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=32722.

htmlize.el was living in the Org contrib directory until Aug 2017:

=====
commit d0ced98943da0e5851ba1145515db27e939bff08
Author: Bastien <bzg@gnu.org>
Date:   Fri Aug 18 09:54:19 2017 +0200

    Delete htmlize.el from Org’s contrib directory

    * lisp/ox-org.el (org-org-publish-to-org):
    * lisp/ox-html.el (org-html-htmlize-generate-css):
    (org-html-fontify-code):
    * lisp/org-agenda.el (org-agenda-write): Throw an error
    requesting the user to install htmlize.el.

    * doc/org.texi (Exporting agenda views, Literal examples):
    Don’t assume htmlize.el is available.

    You need to install it from https://github.com/hniksic/emacs-htmlize

    See https://github.com/hniksic/emacs-htmlize/issues/7 for this issue.

diff --git a/lisp/ox-html.el b/lisp/ox-html.el
--- a/lisp/ox-html.el
+++ b/lisp/ox-html.el
@@ -1746,1 +1747,1 @@
-  (require 'htmlize)
+      (error "Please install htmlize from
https://github.com/hniksic/emacs-htmlize"))
=====

That change lived in Org master branch (devel) for a while, then
propagated to Org 9.1 and eventually to Emacs 26.1 release. So users
switching from Emacs 25.3 to 26.1. would see as an abrupt change, when
in fact it's just that htmlize.el got removed from contrib and instead
started being correctly referenced to its source on GitHub.

> 2.  Is Org part of "Emacs core"?  I didn't think that was how "Emacs
> core" was defined, but I may be wrong.  It is officially part of Emacs,
> of course.  So is there a distinction between "part of Emacs" and "Emacs
> core"?

Sorry, please don't quote me on that. By "Emacs core", I meant all
packages that you can get from an Emacs installation without having to
fetch one from internet.

> Are there any other "time bombs"
> in Org that we should be concerned about?

Not that I know of. I think this is the only one.

> a.  That is not the originally stated problem.

Yes, I know. The original problem was GitHub, and it snowballed to
"htmlize cannot be fetched from outside Emacs". It's just so happens
that someone noticed it and filed a bug report. In Emacs 25.3, if
htmlize weren't installed, it would have thrown an error when (require
'htmlize) got executed (or not, if people were using org-plus-contrib
which htmlize was part of, until Aug 2017).

> 4.  I'm certainly in favor of using built-in libraries as much as
> possible.  If htmlfontify is a better or equivalent solution, and
> someone's willing to write the code and ensure there are no regressions,
> that would be great, because it would save users from having to manually
> install other packages to get expected functionality.

+1

> 5.  Having a passing familiarity with the complexity of the Org code
> base, I am concerned about the potential for regressions in
> functionality and usability.  I'm also a bit disappointed to see this
> burden potentially thrust upon Nicolas and other Org maintainers, to
> replace working code that's existed for over 6 years, for little-to-no
> technical reason.  Their time available for working on Org is very
> valuable.

That's my concern too. And I think that it's mainly Nicolas, and in
another thread, he mentioned that he will do this refactoring, but
without an urgency (which makes sense).

  reply	other threads:[~2018-09-21 21:45 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <531a7f93882cc0d5d09700457d06ea082f47569b.camel@flqt.fr>
2018-09-13  3:35 ` bug#32722: 26.1; Org-publish depend on non-free platform ? Richard Stallman
     [not found] ` <83zhwmlnst.fsf@gnu.org>
2018-09-13  7:33   ` Nicolas Floquet
     [not found] ` <E1g0IPY-000465-4I@fencepost.gnu.org>
2018-09-13 15:26   ` Glenn Morris
     [not found]   ` <tqd0thctb6.fsf@fencepost.gnu.org>
2018-09-13 15:36     ` Kaushal Modi
2018-09-13 21:58       ` Samuel Wales
2018-09-13 21:58       ` Samuel Wales
2018-09-13 21:58       ` bug#32722: " Samuel Wales
2018-09-14  2:55       ` Richard Stallman
     [not found]       ` <E1g0eGJ-0002Ws-Gt@fencepost.gnu.org>
2018-09-14  6:04         ` Eli Zaretskii
2018-09-16 12:30           ` Adam Porter
2018-09-17 20:21             ` Nick Dokos
2018-09-18 13:12               ` Bernt Hansen
     [not found]         ` <83sh2ck42e.fsf@gnu.org>
2018-09-18  9:37           ` Nicolas Goaziou
2018-09-18 11:49             ` Eli Zaretskii
2018-09-18 14:21               ` Nick Dokos
2018-09-18 14:49                 ` Kaushal Modi
     [not found]             ` <83zhwff2k8.fsf@gnu.org>
2018-09-18 12:14               ` bug#32722: " Robert Klein
2018-09-19  3:41                 ` Richard Stallman
2018-09-19  7:22                 ` Eli Zaretskii
2018-09-18 14:43             ` Nicolas Floquet
     [not found]             ` <dbc243f2281e5b97b59d543f8e29559b1ffdc38f.camel@flqt.fr>
2018-09-18 16:37               ` bug#32722: " Robert Horn
     [not found]               ` <m3tvmm4v8m.fsf@panix.com>
2018-09-18 16:53                 ` Eli Zaretskii
     [not found]                 ` <83efdqg32d.fsf@gnu.org>
2018-09-18 18:53                   ` bug#32722: " Robert Klein
2018-09-18 20:41                   ` Glenn Morris
     [not found]                   ` <vain32bksv.fsf_-_@fencepost.gnu.org>
2018-09-18 21:07                     ` Nicolas Goaziou
2018-09-19  3:39                     ` Richard Stallman
     [not found]                     ` <87lg7yfraw.fsf@nicolasgoaziou.fr>
2018-09-18 22:08                       ` bug#32722: " Amin Bandali
2018-09-19 21:16                         ` bug#32722: " Kaushal Modi
2018-09-20  1:54                           ` Richard Stallman
2018-09-20  5:43                           ` Eli Zaretskii
     [not found]                           ` <83h8ikenbc.fsf@gnu.org>
2018-09-20  8:13                             ` bug#32722: " Robert Klein
2018-09-20 19:17                             ` Nicolas Goaziou
2018-09-20 23:17                               ` Adam Porter
2018-09-21  0:39                                 ` Kaushal Modi
2018-09-21  1:54                                   ` Adam Porter
2018-09-21 21:30                                     ` Kaushal Modi [this message]
     [not found]                             ` <87in30geqo.fsf@nicolasgoaziou.fr>
2018-09-20 20:50                               ` Eli Zaretskii
     [not found]                               ` <835zyzevvt.fsf@gnu.org>
2018-10-22 13:13                                 ` Nicolas Goaziou
     [not found]                                 ` <87d0s2yvi4.fsf@nicolasgoaziou.fr>
2018-10-22 13:48                                   ` Eli Zaretskii
     [not found]                           ` <E1g2oAo-0000ao-2k@fencepost.gnu.org>
2018-09-20  4:24                             ` Eli Zaretskii
2018-09-20 10:43                             ` bug#32722: " Joe Corneli
     [not found]                             ` <CAN+qofnbqV8MT0wmKNokXv0S8f9PnXcpK1RdsH-8Zb2i8ckNPw@mail.gmail.com>
2018-09-21  2:13                               ` Richard Stallman
2018-09-21 10:22                                 ` Joe Corneli
     [not found]                                 ` <m2zhwbup3a.fsf@dhcp-90-218.inf.ed.ac.uk>
2018-09-22  1:52                                   ` Richard Stallman
2018-09-19  3:39                       ` Richard Stallman
2018-09-19  7:02                       ` Eli Zaretskii
2018-09-20  5:21                         ` Adam Porter
     [not found]                       ` <831s9qezqo.fsf@gnu.org>
2018-09-20 17:42                         ` Nicolas Goaziou
     [not found]                         ` <87muscgj51.fsf@nicolasgoaziou.fr>
2018-09-20 20:48                           ` Eli Zaretskii
2018-09-21  2:18                           ` Richard Stallman
     [not found]                     ` <E1g2TLB-000734-Os@fencepost.gnu.org>
2018-09-19  7:04                       ` Eli Zaretskii
     [not found]                       ` <83zhwedl2v.fsf@gnu.org>
2018-09-20  1:50                         ` Richard Stallman
     [not found]                         ` <E1g2o79-0008S6-UV@fencepost.gnu.org>
2018-09-20  4:22                           ` Eli Zaretskii
     [not found]                           ` <83lg7wer1z.fsf@gnu.org>
2018-09-21  2:18                             ` Richard Stallman
     [not found]                             ` <E1g3B16-0004U1-2g@fencepost.gnu.org>
2019-05-16 15:16                               ` Nicolas Floquet
     [not found]                               ` <c20679912a5c2644bf6e62a65b7c4da118d988f5.camel@flqt.fr>
2019-05-16 15:25                                 ` Nicolas Floquet
     [not found]                                 ` <3554abc2df1167f5b052b021670bbd09fae6dd6f.camel@flqt.fr>
2019-06-09 16:54                                   ` bug#32722: " Amin Bandali
     [not found]                   ` <20180918205329.2e12c16f@happy.intern.roklein.de>
2018-09-18 18:57                     ` bug#32722: bug#32722: " Kaushal Modi
2018-09-18 19:29                       ` Kaushal Modi
2018-09-19  3:39                         ` Richard Stallman
2018-09-19  3:39                     ` Richard Stallman
2018-09-25 23:04                     ` Richard Stallman
2018-09-19  3:40                 ` Richard Stallman
2018-09-24  9:54                 ` Nicolas Floquet
2018-09-19  3:41               ` Richard Stallman
2018-09-19 12:29                 ` Nicolas Goaziou
     [not found]                 ` <87h8ilfz6o.fsf@nicolasgoaziou.fr>
2018-09-20  1:50                   ` Richard Stallman
     [not found]                   ` <E1g2o6s-0008PG-JL@fencepost.gnu.org>
2018-09-20 12:40                     ` Nicolas Goaziou
2018-09-20 12:59                     ` Bingo UV
2018-09-21  2:13                       ` Richard Stallman
2018-09-21 19:21                         ` bug#32722: bug#32722: " Leslie Watter
     [not found]                         ` <CAKoz4gnvbc1tTGW57c_7cVN0gF2KrgAjYRT+kwp29fMOYX3RCA@mail.gmail.com>
2018-09-23  1:38                           ` Richard Stallman
     [not found]                           ` <E1g3tLf-0003Es-KN@fencepost.gnu.org>
2018-09-29 14:53                             ` Nicolas Goaziou
     [not found]                             ` <87mus0z773.fsf@nicolasgoaziou.fr>
2018-09-29 15:09                               ` Eli Zaretskii
     [not found]                               ` <83in2o5oi3.fsf@gnu.org>
2018-09-29 18:33                                 ` Nicolas Goaziou
2018-09-29 18:58                                   ` Eli Zaretskii
     [not found]                                   ` <83ftxs5dvw.fsf@gnu.org>
2018-09-29 23:35                                     ` bug#32722: " Kyle Meyer
2018-09-30 11:43                                       ` Eli Zaretskii
     [not found]                                       ` <83y3bj43d8.fsf@gnu.org>
2020-05-24  8:03                                         ` bug#32722: [O] " Bastien
2018-09-30  9:27                               ` Michael Albinus
2018-10-22  9:39                                 ` Nicolas Goaziou
     [not found]                                 ` <87r2giz5ey.fsf@nicolasgoaziou.fr>
2018-10-22 11:07                                   ` Michael Albinus
     [not found]                                   ` <871s8is0i6.fsf@gmx.de>
2018-10-22 13:17                                     ` Nicolas Goaziou
     [not found]                     ` <87worggx5b.fsf@nicolasgoaziou.fr>
2018-09-20 14:31                       ` Michael Albinus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAFyQvY3ppssUrUA-O5yC4ytrQ_defd7HS41S3yPKajfAJJU+dQ@mail.gmail.com \
    --to=kaushal.modi@gmail.com \
    --cc=adam@alphapapa.net \
    --cc=emacs-orgmode@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).