[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [AUCTeX-devel] [PATCH v3]: Merge /preview/ into top-level

From: Matthew Leach
Subject: Re: [AUCTeX-devel] [PATCH v3]: Merge /preview/ into top-level
Date: Tue, 18 Nov 2014 23:18:35 +0000
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux)

Mosè Giordano <address@hidden> writes:

> Hi Matthew,

Hi Mosè,

> 2014-11-18 21:50 GMT+01:00 Matthew Leach <address@hidden>:
>> Hi folks,
>> Here is v3 of the preview-merge patch.  This should have taken care of
>> the bits that Mosè found (thanks!).
> I'd like to review this patch, but I'm a bit busy for the time being.
> If someone anticipates me in doing this is really welcome!
>> I have tested most release makefile
>> targets except:
>> * windows-package
>>   - I don't have a windows machine to build and test with.
> You don't need Windows at all, only an Emacs package for Windows.  The
> comments about the release-related rules detail the variables you need
> to set.

Ah, interesting, I will look into this.  I'm guessing I'd still need a
Windows machine to test the package though?

>> * www-doc
>>   - I'm not sure what <patch-file> should be.
> It's a well-kept secret among the AUCTeX developers cabal, but for
> future reference I'm going to share it with you ;-)   Anyway, this
> rule shouldn't be related to the structure of the preview stuff.

Indeed, I don't see how these rules should be affected, but it's good to
be thorough :-).  Thanks for the patch!  Is there a reason it's not kept
in the repo?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]