fletcher / MultiMarkdown-4

This project is now deprecated. Please use MultiMarkdown-6 instead!
https://github.com/fletcher/MultiMarkdown-5
Other
306 stars 59 forks source link

LyX 2.1 support, including Beamer changes, Abbreviation support, leak correctoin #75

Closed crcowan closed 10 years ago

crcowan commented 10 years ago

LyX support has been updated (and tested) to support changes to the LyX 2.1 file structure. Utilization of the new (much improved) beamer support in LyX. Support for the new abbreviation capability. And a memory leak correction.

fletcher commented 10 years ago

Sorry for the delay...

I pulled these changes in, but when I run the make tests, I get errors when testing lyx.

(I had to manually merge the changes, so the commit messages are lost).

crcowan commented 10 years ago

Fletcher:

Thanks. I will take a look but it will probably have to wait until the weekend. When you say "run the make tests" do you mean the normal test run and comparison? When I submitted this pull request, I believe I also update the test library with new ".lyx" versions. Did all the compares fail or just one or two?

On Tue, Aug 19, 2014 at 8:50 AM, Fletcher T. Penney < notifications@github.com> wrote:

Sorry for the delay...

I pulled these changes in, but when I run the make tests, I get errors when testing lyx.

(I had to manually merge the changes, so the commit messages are lost).

— Reply to this email directly or view it on GitHub https://github.com/fletcher/MultiMarkdown-4/pull/75#issuecomment-52628004 .

fletcher commented 10 years ago

Oops. Fixed the test suite, and now it passes.

make test-memory-lyx shows a couple of potential errors, however.

On 8/19/14, 9:05 AM, crcowan wrote:

Fletcher:

Thanks. I will take a look but it will probably have to wait until the weekend. When you say "run the make tests" do you mean the normal test run and comparison? When I submitted this pull request, I believe I also update the test library with new ".lyx" versions. Did all the compares fail or just one or two?

On Tue, Aug 19, 2014 at 8:50 AM, Fletcher T. Penney < notifications@github.com> wrote:

Sorry for the delay...

I pulled these changes in, but when I run the make tests, I get errors when testing lyx.

(I had to manually merge the changes, so the commit messages are lost).

— Reply to this email directly or view it on GitHub

https://github.com/fletcher/MultiMarkdown-4/pull/75#issuecomment-52628004 .

— Reply to this email directly or view it on GitHub https://github.com/fletcher/MultiMarkdown-4/pull/75#issuecomment-52630097.

Fletcher T. Penney fletcher@fletcherpenney.net