Re: CI for "Option handling: add documentation URLs"

2024-03-15 Thread Mark Wielaard
Hi YunQiang Su, On Fri, Mar 15, 2024 at 03:33:28PM +0800, YunQiang Su wrote: > Great work. The CI works well now: it blames me ;) > https://builder.sourceware.org/buildbot/#/builders/269/builds/3846 > > When I add '-mstrict-align' option to MIPS, > the riscv.opt.urls, sysv4.opt.urls,

Re: CI for "Option handling: add documentation URLs"

2024-03-15 Thread YunQiang Su
Great work. The CI works well now: it blames me ;) https://builder.sourceware.org/buildbot/#/builders/269/builds/3846 When I add '-mstrict-align' option to MIPS, the riscv.opt.urls, sysv4.opt.urls, xtensa.opt.urls are changed also. (why they are effected? So what's the best practice for this

Re: CI for "Option handling: add documentation URLs"

2024-03-05 Thread Mark Wielaard
On Tue, Mar 05, 2024 at 08:34:31AM -0500, David Malcolm wrote: > > I committed that patch, but was not fast enough actually enabling the > > buildbot and missed another fixlet needed first. > > > > OK, to push the attached regeneration patch? > > Yes Thanks, pushed. And now also pushed the

Re: CI for "Option handling: add documentation URLs"

2024-03-05 Thread David Malcolm
On Tue, 2024-03-05 at 13:06 +0100, Mark Wielaard wrote: > Hi, > > On Mon, 2024-03-04 at 08:48 -0500, David Malcolm wrote: > > > I have now regenerated the patch to also include the new avr > > > mfuse- > > > add change. It would be nice to get this committed so we can turn > > > on the > > >

Re: CI for "Option handling: add documentation URLs"

2024-03-05 Thread Mark Wielaard
Hi, On Mon, 2024-03-04 at 08:48 -0500, David Malcolm wrote: > > I have now regenerated the patch to also include the new avr mfuse- > > add change. It would be nice to get this committed so we can turn on the > > automatic checker. > > Please go ahead with that. I committed that patch, but was

Re: CI for "Option handling: add documentation URLs"

2024-03-04 Thread David Malcolm
On Sun, 2024-03-03 at 21:04 +0100, Mark Wielaard wrote: > Hi, > > On Sat, Feb 24, 2024 at 06:42:58PM +0100, Mark Wielaard wrote: > > On Thu, Feb 22, 2024 at 11:57:50AM +0800, YunQiang Su wrote: > > > Mark Wielaard 于2024年2月19日周一 06:58写道: > > > > So, I did try the regenerate-opt-urls locally, and

Re: CI for "Option handling: add documentation URLs"

2024-03-03 Thread Mark Wielaard
Hi, On Sat, Feb 24, 2024 at 06:42:58PM +0100, Mark Wielaard wrote: > On Thu, Feb 22, 2024 at 11:57:50AM +0800, YunQiang Su wrote: > > Mark Wielaard 于2024年2月19日周一 06:58写道: > > > So, I did try the regenerate-opt-urls locally, and it did generate the > > > attached diff. Which seems to show we

Re: CI for "Option handling: add documentation URLs"

2024-02-24 Thread Mark Wielaard
Hi, On Thu, Feb 22, 2024 at 11:57:50AM +0800, YunQiang Su wrote: > Mark Wielaard 于2024年2月19日周一 06:58写道: > > So, I did try the regenerate-opt-urls locally, and it did generate the > > attached diff. Which seems to show we really need this automated. > > > > Going over the diff. The

Re: CI for "Option handling: add documentation URLs"

2024-02-21 Thread YunQiang Su
Mark Wielaard 于2024年2月19日周一 06:58写道: > > Hi David, > > On Thu, Jan 04, 2024 at 09:57:09AM -0500, David Malcolm wrote: > > I've pushed the .opt.urls patch kit to gcc trunk [1], so hopefully the > > CI check you wrote can go live now. > > And then I was on vacation myself and forgot. I am sorry. >

Re: CI for "Option handling: add documentation URLs"

2024-02-19 Thread Mark Wielaard
On Sun, 2024-02-18 at 23:58 +0100, Mark Wielaard wrote: > So I think the regenerate-opt-urls check does work as intended. So > lets automate it, because it looks like nobody regenerated the > url.opts after updating the documentation. > > But we should first apply this diff. Could you double

Re: CI for "Option handling: add documentation URLs"

2024-02-18 Thread Mark Wielaard
Hi David, On Thu, Jan 04, 2024 at 09:57:09AM -0500, David Malcolm wrote: > I've pushed the .opt.urls patch kit to gcc trunk [1], so hopefully the > CI check you wrote can go live now. And then I was on vacation myself and forgot. I am sorry. So, I did try the regenerate-opt-urls locally, and it

CI for "Option handling: add documentation URLs"

2024-01-04 Thread David Malcolm
On Fri, 2023-12-15 at 01:59 +0100, Mark Wielaard wrote: > Hi David, > > On Thu, Dec 14, 2023 at 10:01:39AM -0500, David Malcolm wrote: > > > Once your patch is in please feel free to sent an email to > > > build...@sourceware.org > > > https://sourceware.org/mailman/listinfo/buildbot > > > And