From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp11.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id UC4fOsLk1mKFDAEAbAwnHQ (envelope-from ) for ; Tue, 19 Jul 2022 19:07:15 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:bcc0::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp11.migadu.com with LMTPS id MJMWOsLk1mIYRAEA9RJhRA (envelope-from ) for ; Tue, 19 Jul 2022 19:07:14 +0200 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id BD066D5AD for ; Tue, 19 Jul 2022 19:07:14 +0200 (CEST) Received: from localhost ([::1]:39390 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oDqgj-0002iw-Gd for larch@yhetil.org; Tue, 19 Jul 2022 13:07:13 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:41836) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oDqau-0005Bo-K6 for emacs-orgmode@gnu.org; Tue, 19 Jul 2022 13:01:16 -0400 Received: from ciao.gmane.io ([116.202.254.214]:45792) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oDqat-0002eu-51 for emacs-orgmode@gnu.org; Tue, 19 Jul 2022 13:01:12 -0400 Received: from list by ciao.gmane.io with local (Exim 4.92) (envelope-from ) id 1oDqas-0006Ha-1u for emacs-orgmode@gnu.org; Tue, 19 Jul 2022 19:01:10 +0200 X-Injected-Via-Gmane: http://gmane.org/ To: emacs-orgmode@gnu.org From: Max Nikulin Subject: Re: [PATCH] ox-latex.el: Unify in one single list Babel and Polyglossia languages alists Date: Wed, 20 Jul 2022 00:01:04 +0700 Message-ID: References: <87sfxiw2jp.fsf@posteo.net> <87czeduxnz.fsf@localhost> <87lesvlvos.fsf@posteo.net> <87bkto83n7.fsf@localhost> <87v8rtno2j.fsf@posteo.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Content-Language: en-US In-Reply-To: <87v8rtno2j.fsf@posteo.net> Received-SPF: pass client-ip=116.202.254.214; envelope-from=geo-emacs-orgmode@m.gmane-mx.org; helo=ciao.gmane.io X-Spam_score_int: 28 X-Spam_score: 2.8 X-Spam_bar: ++ X-Spam_report: (2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_ADSP_CUSTOM_MED=0.001, FORGED_GMAIL_RCVD=1, FORGED_MUA_MOZILLA=2.309, FREEMAIL_FORGED_FROMDOMAIN=0.249, FREEMAIL_FROM=0.001, HEADER_FROM_DIFFERENT_DOMAINS=0.249, NICE_REPLY_A=-0.001, NML_ADSP_CUSTOM_MED=0.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: emacs-orgmode@gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "General discussions about Org-mode." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-orgmode-bounces+larch=yhetil.org@gnu.org Sender: "Emacs-orgmode" X-Migadu-Flow: FLOW_IN X-Migadu-To: larch@yhetil.org X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1658250434; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=g4odYbSH3ffrhvRBLxcdnw1KnPLhAHIuOWKdReWX//I=; b=Q+enn049ZFyDnbZbIfn0O7q/jCS1spI3ziir+fQKvE813pvhcO7vJSszSEVfrZ+Tqtrl+z oqxgm/5zqfyeGcZtXQMlMinopx/HIerp8xz+tHJ2CngPc93OQdBwG66FKstHnz364c+QD/ nzLKcmll1ZvKr3VZgVL6O4O3fY1ABOg49toJgnslofXF2ZrFl+ZzgrgBQDJfMX7+m8pi2Z vjeUAEFvINgG5d+Kic9mD3hVMQ2TD9g9QbY8PY9H1KdRpj59g38Uy5p8/mqM/Q+NK5xV7g 2uF4/Jgj8PaTzBmawkAOpZT/CRuEsnzHbP7j79mtccwAjHfWN4dky2mMMQHFww== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1658250434; a=rsa-sha256; cv=none; b=h/cD4S8TtALWkGKjuhQ9Ulz77O9GzcYtUXxoxBfeuHgT/UEw24QNwNcLSgxmRIHQ0nZljZ CaNUu8h+PSpdiT/84Oynf7ayi9ipfXKaRvyvsxbBgNJFLZyP74IONleT+eTQ5eBLWQ73FY fnC/pJRwpeyGcnDY3LL3JPDXUOYic4twGKd+KkI8P+iVBfs0hpGvIrw48MTf74x/BsCCjR EwtfvXywGbm6KsCAPpsRcgSgEgIe/vMFCDOMOPwc8lr2TK+IprJssS6ZLFRZxZhdMA93f3 6uyR8qUYCmLwZILpVXVPyr0J0rpjnvqEJ1bdJAxPULaa2uhqqDT3bL4Y1S5tWA== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=none; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=gmail.com (policy=none); spf=pass (aspmx1.migadu.com: domain of "emacs-orgmode-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="emacs-orgmode-bounces+larch=yhetil.org@gnu.org" X-Migadu-Spam-Score: 1.97 Authentication-Results: aspmx1.migadu.com; dkim=none; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=gmail.com (policy=none); spf=pass (aspmx1.migadu.com: domain of "emacs-orgmode-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="emacs-orgmode-bounces+larch=yhetil.org@gnu.org" X-Migadu-Queue-Id: BD066D5AD X-Spam-Score: 1.97 X-Migadu-Scanner: scn0.migadu.com X-TUID: kZtXTQv4rIJ8 On 19/07/2022 22:01, Juan Manuel Macías wrote: > + (replace-match > + (mapconcat (lambda (option) (if (equal "AUTO" option) language option)) > + (cond ((member language options) (delete "AUTO" options)) > + ((member "AUTO" options) options) > + (t (append options (list language)))) > + ", ") In my opinion this code should not attempt to be excessively clever. If user skipped AUTO then do not append language. Test for duplicated options is redundant as well. Such cases may still be a reason to issue a warning (e.g. by `org-lint'). On the other hand I would consider adding babel by default without explicit header. To suppress loading users may add #+latex_header: % \usepackage{babel} I like that you decided to avoid inventing of a DSL just to have org-like options that are translated to to a couple of preamble commands. From my point of view it would not help novices and would make things more complicated for experienced LaTeX users. > #+latex_class: article > #+latex_class_options: [11pt,draft,AUTO] > #+LaTeX_Header: \usepackage{babel} > #+language: ru While AUTO is suitable for \usepackage{babel} and \babelprovide, for class option the placeholder should be clearly associated with babel, e.g. BABEL_LANG instead of AUTO. I am not familiar enough with setting used to generate preview of equations or images from LaTeX source blocks, so I am not completely sure that suggested changes do not affect these features in some negative way.