unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Dmitry Gutov <dgutov@yandex.ru>
To: "João Távora" <joaotavora@gmail.com>
Cc: Mark Oteiza <mvoteiza@udel.edu>,
	Lele Gaifax <lele@metapensiero.it>,
	Stefan Monnier <monnier@iro.umontreal.ca>,
	emacs-devel@gnu.org
Subject: Re: Three Flymake backends Was Re: Two issues with the new Flymake
Date: Mon, 13 Nov 2017 02:23:04 +0200	[thread overview]
Message-ID: <b037696f-4957-7f43-7929-7268f132cbc5@yandex.ru> (raw)
In-Reply-To: <871slbr7ft.fsf@gmail.com>

On 11/6/17 1:08 PM, João Távora wrote:

> Then probably a good idea to do what flycheck does because gremlins.

Sure.

>>> Hey I didn't spend more than 5 minutes with it :-)
>>
>> However long, I'm glad you did. Now, the version I've sent is pretty
>> much working.
>>
>> Will you push it to emacs-26?
> 
> I think you should test it a tiny bit more and then you push it :-)

It seems to be working well, but I'm getting bogged down by minor details.

Do we add defcustoms for the program name and the config file name? Do 
we really need the ruby-flymake-command defcustom? I doubt there is an 
alternative program that gives the same output as 'ruby -wc'.

>> Where will it live?
> 
> ruby-mode.el so it at least locally shares some code with ruby-flymake.

OK, code sharing will be step two.

>> How/when will Flymake choose between rubocop-flymake and ruby-flymake?
> 
> Flymake can use both at the same time. Just
> 
>     (add-hook 'flymake-diagnostic-functions 'ruby-flymake nil t)
>     (add-hook 'flymake-diagnostic-functions 'rubocop-flymake nil t)
> 
> The user can remove-hook if he wants to.
> 
> If rubocop does everything "ruby -w" does and more,

It does.

> then maybe a single
> backend. One that uses a ruby-flymake-use-rubocop-if-available
> defcustom, and then checks for (executable-find "rubocop").

Do we check for (executable-find "rubocop") once inside the major mode 
function, or every time the checker is called?

In the latter case we'll have a function ruby-flymake-auto calling one 
or the other.



      reply	other threads:[~2017-11-13  0:23 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-03  9:50 Two issues with the new Flymake Lele Gaifax
2017-11-03 12:33 ` Stefan Monnier
2017-11-03 14:07   ` Lele Gaifax
2017-11-03 16:59     ` João Távora
2017-11-03 17:15       ` Stefan Monnier
2017-11-03 20:17 ` Three Flymake backends Was " João Távora
2017-11-04 15:30   ` Stefan Monnier
2017-11-04 23:17     ` João Távora
2017-11-05 12:50   ` Dmitry Gutov
2017-11-05 12:59     ` João Távora
2017-11-05 13:04       ` Dmitry Gutov
2017-11-05 13:22         ` João Távora
2017-11-05 20:14           ` Dmitry Gutov
2017-11-05 21:05             ` João Távora
2017-11-05 23:56               ` Dmitry Gutov
2017-11-06  9:48                 ` João Távora
2017-11-06 10:35                   ` Dmitry Gutov
2017-11-06 11:08                     ` João Távora
2017-11-13  0:23                       ` Dmitry Gutov [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b037696f-4957-7f43-7929-7268f132cbc5@yandex.ru \
    --to=dgutov@yandex.ru \
    --cc=emacs-devel@gnu.org \
    --cc=joaotavora@gmail.com \
    --cc=lele@metapensiero.it \
    --cc=monnier@iro.umontreal.ca \
    --cc=mvoteiza@udel.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).