Makefile colorizing improvement: colorize at the beginning of line, handling the '\' symbol and other.

[Alexander Fadeev (fadeevab)]

Pull request is created into TextMate repository as well: textmate/make.tmbundle#15

1). Colorize when there is an expression at the beginning of line.
2). Handle '\' at the end of prerequisites (the next line should be colorized as a continuation of previous).
3). Colorize (origin|flavor).
4). Fix colorizing of built-in which stands after a common variable.

[Microsoft Contribution License Agreements (msftclas)]

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

[Martin Aeschlimann (aeschli)]

Alexander Fadeev (@fadeevab) Sorry for my previous comment, now I see your pull request.
Give it a bit time for the guys at textmate/make.tmbundle to look into it.
If nothing happens there we can think again what to do and if we make an exception and move ahead.

[Microsoft Contribution License Agreements (msftclas)]

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

[Alexander Fadeev (fadeevab)]

Martin Aeschlimann (@aeschli), Sure, thank you! It would be great if somebody review my pull request more carefully. But actually, current makefile colorizing couldn't be worse, so I believe this patch makes one better. At least I tried to test it carefully, that's why I recreated pull request with additional fixes. And also several fixes will come later, because some cases are still not covered.

[Microsoft Contribution License Agreements (msftclas)]

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

[Alexander Fadeev (fadeevab)]

(re-opened because it's closed accidentally from mobile phone)

[Alexander Fadeev (fadeevab)]

Nothing happens in the referred pull request. Wait more?

[Alexander Fadeev (fadeevab)]

1. There is some movement in the referred review at TextMate Makefile colorizing improvement: colorize at the beginning of line, handling the '\' symbol and other. textmate/make.tmbundle#15.
2. I actualized makefile tests.
   

[Alexander Fadeev (fadeevab)]

A moment ago both builds were ok. Now AppVeyor build failed, I'm sure it doesn't depend on update.

npm ERR! code EBADF
npm ERR! errno -4083
npm ERR! syscall fstat
npm ERR! EBADF: bad file descriptor, fstat

[Alexander Fadeev (fadeevab)]

Review is stuck

[Alexander Fadeev (fadeevab)]

Martin Aeschlimann (@aeschli), TextMate review of Makefile syntax is not moving on. What could we do in such case: could we submit?

[Martin Aeschlimann (aeschli)]

Let's be patient one more week or so. Creating our own fork is not really what we want as we don't have the resources to maintain it. So given that you got some response on your PR I'd rather wait a bit more.

[Alexander Fadeev (fadeevab)]

Right, I just don't want this patch would be sank into oblivion.

[Alexander Fadeev (fadeevab)]

Martin Aeschlimann (@aeschli), 3 months are passed. I give up to push TextMate team. You can see there is a lot of my comments, and also I divided my commit into 4 commits there. There is only one reviewer and he doesn't take an action.

[Martin Aeschlimann (aeschli)]

What we can do is that we use your fork (https://github.com/fadeevab/make.tmbundle/) as the new input for the make grammar. Would that be ok for you?

• you'd have to make sure to keep master in a working state
• avoid diverging from the original so that we eventually can unify them again.
• new issues would be filed against your repo

Ideally this is only a temporary solution.

Let me know when your repo is ready.

[Alexander Fadeev (fadeevab)]

Just let's do this. I agree to send new pull request to the original repo to synchronize our repos, so we can eventually unify them. Also I'll try to take care about new issues, still maybe it worth to duplicate the issues to the original repo.

[Alexander Fadeev (fadeevab)]

Martin Aeschlimann (@aeschli), Any chances? :)

[Martin Aeschlimann (aeschli)]

Thanks Alexander Fadeev (@fadeevab) !

[Martin Aeschlimann (aeschli)]

Alexander Fadeev (@fadeevab) I'm now consuming your repo. To simplify things, can you add an MIT LICENSE.md file to your repo?
I'd suggest to append the licence mentioned in https://github.com/fadeevab/make.tmbundle/blob/master/README.mdown to that

[Alexander Fadeev (fadeevab)]

Martin Aeschlimann (@aeschli), Done