On Sun, May 3, 2015 at 11:00 AM, Enrico Forestieri <for...@lyx.org> wrote:
> On Sun, May 03, 2015 at 01:52:33AM -0400, Scott Kostyshak wrote:
>
>> On Sat, May 2, 2015 at 8:02 PM, Enrico Forestieri <for...@lyx.org> wrote:
>> > On Sat, May 02, 2015 at 07:03:53PM -0400, Scott Kostyshak wrote:
>> >>
>> >> An alternative would be to add a parameter to LaTeX::LaTeX() that says
>> >> whether to do a fresh compile.
>> >
>> > This alternative seems the most logical one to me. Add a bool parameter
>> > (clean_start or similar) which is false by default to tell the
>> > constructor to start afresh by removing previous leftovers.
>>
>> OK, attached is an updated patch.
>
> What about renaming deleteFilesOnError as removeAuxiliaryFiles?

I agree that changing the name would make sense. But in that function
we remove the main output file (e.g. PDF) I believe so I don't know if
"auxiliary" is fitting. How about removeTempFiles or removeCachedFiles
or removeLaTeXFiles?

Scott

Reply via email to