Re: [AUCTeX-devel] Management of package options [Was: Re: Add `unit' type for the parser in siunitx.el]

2013-04-02 Thread Tassilo Horn
Tassilo Horn  writes:

 I'm sending patches implementing what Tassilo suggested.  In
 `LaTeX-arg-usepackage' I replaced `match-string' with
 `buffer-substring-no-properties' because I need to remove
 properties, `match-string-no-properties' requires Emacs 21.3 and I
 don't know whether XEmacs has it.
>>>
>>> It does.
>>
>> So can I use `match-string-no-properties'?
>
> Yes.

Ups, I've just found out that the latest non-beta XEmacs 21.4.22
released in 2009 doesn't have `match-string-no-properties', please use
use `match-string' and `remove-text-properties'.

Bye,
Tassilo


___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


Re: [AUCTeX-devel] Management of package options [Was: Re: Add `unit' type for the parser in siunitx.el]

2013-04-02 Thread Tassilo Horn
Mosè Giordano  writes:

Hi Mosè,

>>> I'm sending patches implementing what Tassilo suggested.  In
>>> `LaTeX-arg-usepackage' I replaced `match-string' with
>>> `buffer-substring-no-properties' because I need to remove
>>> properties, `match-string-no-properties' requires Emacs 21.3 and I
>>> don't know whether XEmacs has it.
>>
>> It does.
>
> So can I use `match-string-no-properties'?

Yes.

> AUCTeX requires Emacs 21, but 21.1 or 21.3 is also acceptable?

21.1 is the first Emacs 21 release, released in the end of 2001.  21.3
is about 10 years old, and I think we can assume that people have at
least that version.  If someone complains, it's easy to use
`match-string' and `remove-text-properties' in place of
`match-string-no-properties'.

Bye,
Tassilo

___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


Re: [AUCTeX-devel] Management of package options [Was: Re: Add `unit' type for the parser in siunitx.el]

2013-04-02 Thread Mosè Giordano
Hi Tassilo,

2013/4/2 Tassilo Horn :
> Mosè Giordano  writes:
>
> Hi Mosè,
>
>> I'm sending patches implementing what Tassilo suggested.  In
>> `LaTeX-arg-usepackage' I replaced `match-string' with
>> `buffer-substring-no-properties' because I need to remove properties,
>> `match-string-no-properties' requires Emacs 21.3 and I don't know
>> whether XEmacs has it.
>
> It does.
So can I use `match-string-no-properties'?  AUCTeX requires Emacs 21,
but 21.1 or 21.3 is also acceptable?

>> In `LaTeX-auto-cleanup', I haven't changed this
>> ---
>>   (add-to-list 'TeX-auto-file
>>(concat
>> (cond ((string-equal "article" style)
>>"art")
>>   ((string-equal "book" style)
>>"bk")
>>   ((string-equal "report" style)
>>"rep")
>>   ((string-equal "jarticle" style)
>>"jart")
>>   ((string-equal "jbook" style)
>>"jbk")
>>   ((string-equal "jreport" style)
>>"jrep")
>>   ((string-equal "j-article" style)
>>"j-art")
>>   ((string-equal "j-book" style)
>>"j-bk")
>>   ((string-equal "j-report" style )
>>"j-rep")
>>   (t style))
>> (cond ((member "11pt" options)
>>"11")
>>   ((member "12pt" options)
>>"12")
>>   (t
>>"10"))) t)
>> ---
>> because I've never understood what's its purpose.
>
> It seems that there are style files like art11.sty and class option
> files like bk11.clo.  My TeX distribution only has bk{10,11,12}.clo, but
> none of the art, rep or j-* files.  But when you google for
> them, you'll find them.
Thanks for you explanation.

Bye,
Mosè Giordano

___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


[AUCTeX-devel] New AUCTeX Git repository (testing phase)

2013-04-02 Thread Tassilo Horn
Hi all,

I've just converted the AUCTeX CVS repository to Git and pushed it to
savannah.  You shouldn't start hacking using the new Git repo right
away.  I'm not completely sure everything's right in there, so I might
need to do another conversion.

I'd very much appreciate any testing you can do to increase my trust in
that the conversion worked out well.  E.g., check out a given tag in
both CVS and Git and test if all the files are equal.  I guess this can
be automated, since there are a lot of tags.

If nobody including me finds something strange, then I'm going to
disable the CVS repository and make the Git one official really soon™.

Bye,
Tassilo


___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


Re: [AUCTeX-devel] Search for LaTeX classes

2013-04-02 Thread Tassilo Horn
Mosè Giordano  writes:

Hi Mosè,

> I'm sending a patch for `latex.el' to make `TeX-arg-document' search
> for LaTeX classes.

Thanks, applied.

Bye,
Tassilo


___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


[AUCTeX-diffs] Changes to auctex/ChangeLog,v

2013-04-02 Thread Tassilo Horn
CVSROOT:/sources/auctex
Module name:auctex
Changes by: Tassilo Horn  13/04/02 07:22:30

Index: ChangeLog
===
RCS file: /sources/auctex/auctex/ChangeLog,v
retrieving revision 5.1863
retrieving revision 5.1864
diff -u -b -r5.1863 -r5.1864
--- ChangeLog   28 Mar 2013 16:25:00 -  5.1863
+++ ChangeLog   2 Apr 2013 07:22:29 -   5.1864
@@ -1,3 +1,7 @@
+2013-03-29  Mosè Giordano  
+
+   * latex.el (TeX-arg-document): Search for LaTeX classes.
+
 2013-03-28  Mosè Giordano  
 
* style/biblatex.el (LaTeX-biblatex-package-options): Correct name

___
auctex-diffs mailing list
auctex-di...@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-diffs


[AUCTeX-diffs] Changes to auctex/latex.el,v

2013-04-02 Thread Tassilo Horn
CVSROOT:/sources/auctex
Module name:auctex
Changes by: Tassilo Horn  13/04/02 07:22:30

Index: latex.el
===
RCS file: /sources/auctex/auctex/latex.el,v
retrieving revision 5.492
retrieving revision 5.493
diff -u -b -r5.492 -r5.493
--- latex.el28 Mar 2013 10:14:46 -  5.492
+++ latex.el2 Apr 2013 07:22:29 -   5.493
@@ -1737,7 +1737,15 @@
 (defun TeX-arg-document (optional &optional ignore)
   "Insert arguments to documentclass.
 OPTIONAL and IGNORE are ignored."
-  (let ((style (completing-read
+  (let* ((TeX-file-extensions '("cls"))
+(search (if (eq TeX-arg-input-file-search 'ask)
+(not (y-or-n-p "Find class yourself? "))
+  TeX-arg-input-file-search))
+(LaTeX-style-list
+ (if search
+ (mapcar 'identity (TeX-search-files-by-type 'texinputs 'global t 
t))
+   LaTeX-style-list))
+(style (completing-read
(concat "Document class: (default " LaTeX-default-style ") ")
LaTeX-style-list))
(options (read-string "Options: "

___
auctex-diffs mailing list
auctex-di...@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-diffs


Re: [AUCTeX-devel] Management of package options [Was: Re: Add `unit' type for the parser in siunitx.el]

2013-04-02 Thread Tassilo Horn
Mosè Giordano  writes:

> Regarding `LaTeX-arg-usepackage', is `TeX-input-file-search' (in first
> `let') a typo?

Oh yes, seems so.

> That variable isn't defined anywhere, should it be
> `TeX-arg-input-file-search'?

Yes.

> If this is the case, why force searching if user doesn't want it?

Hm, I think for \usepackage it's appropriate to search.  Maybe it should
be bound to

  (or TeX-arg-input-file-search 'ask)

in `LaTeX-arg-usepackage'?  Then a user who set
TeX-arg-input-file-search to nil would be queried.

Bye,
Tassilo


___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel


Re: [AUCTeX-devel] Management of package options [Was: Re: Add `unit' type for the parser in siunitx.el]

2013-04-02 Thread Tassilo Horn
Mosè Giordano  writes:

Hi Mosè,

> I'm sending patches implementing what Tassilo suggested.  In
> `LaTeX-arg-usepackage' I replaced `match-string' with
> `buffer-substring-no-properties' because I need to remove properties,
> `match-string-no-properties' requires Emacs 21.3 and I don't know
> whether XEmacs has it.

It does.

> In `LaTeX-auto-cleanup', I haven't changed this
> ---
>   (add-to-list 'TeX-auto-file
>(concat
> (cond ((string-equal "article" style)
>"art")
>   ((string-equal "book" style)
>"bk")
>   ((string-equal "report" style)
>"rep")
>   ((string-equal "jarticle" style)
>"jart")
>   ((string-equal "jbook" style)
>"jbk")
>   ((string-equal "jreport" style)
>"jrep")
>   ((string-equal "j-article" style)
>"j-art")
>   ((string-equal "j-book" style)
>"j-bk")
>   ((string-equal "j-report" style )
>"j-rep")
>   (t style))
> (cond ((member "11pt" options)
>"11")
>   ((member "12pt" options)
>"12")
>   (t
>"10"))) t)
> ---
> because I've never understood what's its purpose.

It seems that there are style files like art11.sty and class option
files like bk11.clo.  My TeX distribution only has bk{10,11,12}.clo, but
none of the art, rep or j-* files.  But when you google for
them, you'll find them.

> If these patches are fine, I will send also patches for style files
> checking package options (they should be only babel, biblatex,
> hyperref, siunitx) and the full ChangeLog entry.

Yes, the changes look good to me.  Then I'll wait for a complete patch
with ChangeLog before committing.

Bye,
Tassilo


___
auctex-devel mailing list
auctex-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/auctex-devel