All of lore.kernel.org
 help / color / mirror / Atom feed
* new: b4 send --reflect
@ 2022-12-06 21:53 Konstantin Ryabitsev
  2022-12-07 16:39 ` Noralf Trønnes
  0 siblings, 1 reply; 8+ messages in thread
From: Konstantin Ryabitsev @ 2022-12-06 21:53 UTC (permalink / raw)
  To: tools; +Cc: Noralf Trønnes

Hi, all:

Final feature before 0.11 (hopefully) -- there is a new test mode for b4 send
that will only email the series to yourself instead of the actual destinations
listed in "To" or "Cc" headers.

This should be particularly useful to anyone using the web submission
endpoint, because this will allow them to see any post-processing that the
remote will do to their patches (such as rewriting From: for DMARC purposes).

If you're using b4 from git, I recommend that you get the latest master so it
can see some testing before the 0.11 release later in the week. 
The main notable new features in 0.11 as compared to 0.10.y are:

- b4 prep --compare-to N: shows a range-diff between your current prep branch
  and the previously submitted revision N

- b4 ty: can be configured to always send mail without requiring the -S switch
  (via setting b4.ty-send-mail=yes)

- b4 send will no longer send the entire series to the addresses found in
  individual commit Cc: trailers; such addresses will only receive the cover
  letter and the patches where they are explicitly Cc'd

- you can now set extra [PATCH] prefixes permanently either via b4 prep
  --set-prefixes or via the cover letter subject line

- internal changes to ensure that we generate 7-bit-clean headers for all
  outgoing mail

- b4 send --reflect: prepares the patches but only sends them to yourself

If all goes well, I will release 0.11 either Thursday or Friday.

Best wishes,
Konstantin

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-06 21:53 new: b4 send --reflect Konstantin Ryabitsev
@ 2022-12-07 16:39 ` Noralf Trønnes
  2022-12-07 18:12   ` Noralf Trønnes
  2022-12-07 18:21   ` Konstantin Ryabitsev
  0 siblings, 2 replies; 8+ messages in thread
From: Noralf Trønnes @ 2022-12-07 16:39 UTC (permalink / raw)
  To: Konstantin Ryabitsev, tools; +Cc: Noralf Trønnes

[-- Attachment #1: Type: text/plain, Size: 1887 bytes --]



Den 06.12.2022 22.53, skrev Konstantin Ryabitsev:
> Hi, all:
> 
> Final feature before 0.11 (hopefully) -- there is a new test mode for b4 send
> that will only email the series to yourself instead of the actual destinations
> listed in "To" or "Cc" headers.
> 
> This should be particularly useful to anyone using the web submission
> endpoint, because this will allow them to see any post-processing that the
> remote will do to their patches (such as rewriting From: for DMARC purposes).
> 

I've always sent the patches to myself for a final check before sending
them off, so this is perfect.

But it doesn't work for me yet:

$ ~/projects/b4/b4.sh send --hide-cover-to-cc --reflect
Converted the branch to 1 messages
---
To: dri-devel@lists.freedesktop.org
    tzimmermann@suse.de
    kernel test robot <lkp@intel.com>
    Noralf Trønnes <noralf@tronnes.org>
---
  [PATCH] drm/gud: Fix missing include
---
Ready to:
  - send the above messages to just noralf@tronnes.org (REFLECT MODE)
  - via web endpoint: https://lkml.kernel.org/_b4_submit

Press Enter to proceed or Ctrl-C to abort
---
Reflecting via web endpoint https://lkml.kernel.org/_b4_submit
Error from endpoint: This service only accepts patches
---
CRITICAL: Was not able to send messages.

$ ~/projects/b4/b4.sh --version
0.11.0-dev-b77ec

Cover letter:

$ git show 6c2812de7efe
commit 6c2812de7efe27951c95c9f736215675009da3fb
Author: Noralf Trønnes <noralf@tronnes.org>
Date:   Wed Dec 7 12:58:01 2022 +0100

    dummy
    Cc: dri-devel@lists.freedesktop.org
    Cc: tzimmermann@suse.de

    --- b4-submit-tracking ---
    # This section is used internally by b4 prep for tracking purposes.
    {
      "series": {
        "revision": 1,
        "change-id": "20221207-gud-missing-include-9ccf56382f8a",
        "base-branch": null
      }
    }

I've attached the 'b4 send --hide-cover-to-cc -o' output.

Noralf.

[-- Attachment #2: b4-gud-missing-include-1.tar.gz --]
[-- Type: application/x-gzip, Size: 1125 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 16:39 ` Noralf Trønnes
@ 2022-12-07 18:12   ` Noralf Trønnes
  2022-12-07 18:21   ` Konstantin Ryabitsev
  1 sibling, 0 replies; 8+ messages in thread
From: Noralf Trønnes @ 2022-12-07 18:12 UTC (permalink / raw)
  To: Konstantin Ryabitsev, tools; +Cc: Noralf Trønnes



Den 07.12.2022 17.39, skrev Noralf Trønnes:
> 
> 
> Den 06.12.2022 22.53, skrev Konstantin Ryabitsev:
>> Hi, all:
>>
>> Final feature before 0.11 (hopefully) -- there is a new test mode for b4 send
>> that will only email the series to yourself instead of the actual destinations
>> listed in "To" or "Cc" headers.
>>
>> This should be particularly useful to anyone using the web submission
>> endpoint, because this will allow them to see any post-processing that the
>> remote will do to their patches (such as rewriting From: for DMARC purposes).
>>
> 
> I've always sent the patches to myself for a final check before sending
> them off, so this is perfect.
> 
> But it doesn't work for me yet:
> 
> $ ~/projects/b4/b4.sh send --hide-cover-to-cc --reflect
> Converted the branch to 1 messages
> ---
> To: dri-devel@lists.freedesktop.org
>     tzimmermann@suse.de
>     kernel test robot <lkp@intel.com>
>     Noralf Trønnes <noralf@tronnes.org>
> ---
>   [PATCH] drm/gud: Fix missing include
> ---
> Ready to:
>   - send the above messages to just noralf@tronnes.org (REFLECT MODE)
>   - via web endpoint: https://lkml.kernel.org/_b4_submit
> 
> Press Enter to proceed or Ctrl-C to abort
> ---
> Reflecting via web endpoint https://lkml.kernel.org/_b4_submit
> Error from endpoint: This service only accepts patches
> ---
> CRITICAL: Was not able to send messages.
> 
> $ ~/projects/b4/b4.sh --version
> 0.11.0-dev-b77ec
> 
> Cover letter:
> 
> $ git show 6c2812de7efe
> commit 6c2812de7efe27951c95c9f736215675009da3fb
> Author: Noralf Trønnes <noralf@tronnes.org>
> Date:   Wed Dec 7 12:58:01 2022 +0100
> 
>     dummy
>     Cc: dri-devel@lists.freedesktop.org
>     Cc: tzimmermann@suse.de
> 
>     --- b4-submit-tracking ---
>     # This section is used internally by b4 prep for tracking purposes.
>     {
>       "series": {
>         "revision": 1,
>         "change-id": "20221207-gud-missing-include-9ccf56382f8a",
>         "base-branch": null
>       }
>     }
> 
> I've attached the 'b4 send --hide-cover-to-cc -o' output.
> 

I wanted to get this patch out so I dropped the --reflect, but I got the
same error.

Noralf.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 16:39 ` Noralf Trønnes
  2022-12-07 18:12   ` Noralf Trønnes
@ 2022-12-07 18:21   ` Konstantin Ryabitsev
  2022-12-07 19:36     ` Noralf Trønnes
  1 sibling, 1 reply; 8+ messages in thread
From: Konstantin Ryabitsev @ 2022-12-07 18:21 UTC (permalink / raw)
  To: Noralf Trønnes; +Cc: tools

On Wed, Dec 07, 2022 at 05:39:40PM +0100, Noralf Trønnes wrote:
> Error from endpoint: This service only accepts patches

Stupidly, I never tested it with a single patch, always with a series.
Please try again.

-K

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 18:21   ` Konstantin Ryabitsev
@ 2022-12-07 19:36     ` Noralf Trønnes
  2022-12-07 19:45       ` Konstantin Ryabitsev
  0 siblings, 1 reply; 8+ messages in thread
From: Noralf Trønnes @ 2022-12-07 19:36 UTC (permalink / raw)
  To: Konstantin Ryabitsev; +Cc: tools, Noralf Trønnes



Den 07.12.2022 19.21, skrev Konstantin Ryabitsev:
> On Wed, Dec 07, 2022 at 05:39:40PM +0100, Noralf Trønnes wrote:
>> Error from endpoint: This service only accepts patches
> 
> Stupidly, I never tested it with a single patch, always with a series.
> Please try again.
> 

Getting closer ;)

$ ~/projects/b4/b4.sh send --hide-cover-to-cc --reflect
Converted the branch to 1 messages
---
To: dri-devel@lists.freedesktop.org
    tzimmermann@suse.de
    kernel test robot <lkp@intel.com>
    Noralf Trønnes <noralf@tronnes.org>
---
  [PATCH] drm/gud: Fix missing include
---
Ready to:
  - send the above messages to just noralf@tronnes.org (REFLECT MODE)
  - via web endpoint: https://lkml.kernel.org/_b4_submit

Press Enter to proceed or Ctrl-C to abort
---
Reflecting via web endpoint https://lkml.kernel.org/_b4_submit
Error from endpoint: Destinations must include a mailing list we recognize.
---
CRITICAL: Was not able to send messages.

I'm getting the same without --reflect

Noralf.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 19:36     ` Noralf Trønnes
@ 2022-12-07 19:45       ` Konstantin Ryabitsev
  2022-12-07 19:54         ` Noralf Trønnes
  0 siblings, 1 reply; 8+ messages in thread
From: Konstantin Ryabitsev @ 2022-12-07 19:45 UTC (permalink / raw)
  To: Noralf Trønnes; +Cc: tools

On Wed, Dec 07, 2022 at 08:36:44PM +0100, Noralf Trønnes wrote:
> > On Wed, Dec 07, 2022 at 05:39:40PM +0100, Noralf Trønnes wrote:
> >> Error from endpoint: This service only accepts patches
> > 
> > Stupidly, I never tested it with a single patch, always with a series.
> > Please try again.
> > 
> 
> Getting closer ;)

You're tripping up all the corner-cases, this is great! :)

> Reflecting via web endpoint https://lkml.kernel.org/_b4_submit
> Error from endpoint: Destinations must include a mailing list we recognize.

One of my goals is to make sure that this service can't be abused to send
spam, so we're running a bunch of checks to verify that it's a legitimate
patch sent to a sane set of destinations. In this particular case, we weren't
expecting the only list to be lists.freedesktop.org (usually there's always a
kernel.org address there somewhere).

Please try again.

-K

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 19:45       ` Konstantin Ryabitsev
@ 2022-12-07 19:54         ` Noralf Trønnes
  2022-12-07 20:07           ` Konstantin Ryabitsev
  0 siblings, 1 reply; 8+ messages in thread
From: Noralf Trønnes @ 2022-12-07 19:54 UTC (permalink / raw)
  To: Konstantin Ryabitsev; +Cc: tools, Noralf Trønnes



Den 07.12.2022 20.45, skrev Konstantin Ryabitsev:
> On Wed, Dec 07, 2022 at 08:36:44PM +0100, Noralf Trønnes wrote:
>>> On Wed, Dec 07, 2022 at 05:39:40PM +0100, Noralf Trønnes wrote:
>>>> Error from endpoint: This service only accepts patches
>>>
>>> Stupidly, I never tested it with a single patch, always with a series.
>>> Please try again.
>>>
>>
>> Getting closer ;)
> 
> You're tripping up all the corner-cases, this is great! :)
> 
>> Reflecting via web endpoint https://lkml.kernel.org/_b4_submit
>> Error from endpoint: Destinations must include a mailing list we recognize.
> 
> One of my goals is to make sure that this service can't be abused to send
> spam, so we're running a bunch of checks to verify that it's a legitimate
> patch sent to a sane set of destinations. In this particular case, we weren't
> expecting the only list to be lists.freedesktop.org (usually there's always a
> kernel.org address there somewhere).
> 
> Please try again.
> 

It's working now, thanks.

Noralf.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: new: b4 send --reflect
  2022-12-07 19:54         ` Noralf Trønnes
@ 2022-12-07 20:07           ` Konstantin Ryabitsev
  0 siblings, 0 replies; 8+ messages in thread
From: Konstantin Ryabitsev @ 2022-12-07 20:07 UTC (permalink / raw)
  To: Noralf Trønnes; +Cc: tools

On Wed, Dec 07, 2022 at 08:54:20PM +0100, Noralf Trønnes wrote:
> It's working now, thanks.

Great. Thanks again for being a willing test subject!

-K

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2022-12-07 20:07 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-06 21:53 new: b4 send --reflect Konstantin Ryabitsev
2022-12-07 16:39 ` Noralf Trønnes
2022-12-07 18:12   ` Noralf Trønnes
2022-12-07 18:21   ` Konstantin Ryabitsev
2022-12-07 19:36     ` Noralf Trønnes
2022-12-07 19:45       ` Konstantin Ryabitsev
2022-12-07 19:54         ` Noralf Trønnes
2022-12-07 20:07           ` Konstantin Ryabitsev

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.