On Thu, Apr 6, 2023 at 6:23 PM Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > > On Thu, 6 Apr 2023 18:17:57 -0700 Alexei Starovoitov wrote: > > On Thu, Apr 6, 2023 at 8:42 AM Jakub Kicinski <kuba@xxxxxxxxxx> wrote: > > > > Yeah. If only... > > > > I'm exclusively using -c. > > > > -M only works with -s, but I couldn't make -s -M work either. > > > > Do you pass the series as a number? > > > > > > Yes, it copy just the numerical ID into the terminal. > > > > > > > but then series_json=$(curl -s $srv/series/$1/) line > > > > doesn't look right, since it's missing "/mbox/" ? > > > > > > That's loading JSON from the patchwork's REST API. > > > > This line still doesn't work for me. > > curl -s https://patchwork.kernel.org/series/736654/ > > returns: > > The page URL requested (<code>/series/736654/</code>) does not exist. > > > > while > > curl -s https://patchwork.kernel.org/series/736654/mbox/ > > returns proper mbox format. > > Check if your git config is right: > > $ git config --get pw.server > https://patchwork.kernel.org/api/1.1/ > > that's where $srv comes from Ahh. All works now! I like the new output. I'll play with it more. Should -M be a default? Any downside?
