AgreeThank you. 

See below 



Leslie Satenstein    
 

    On Thursday, January 18, 2024 at 01:35:13 p.m. EST, kevin <ke...@scrye.com> 
wrote:  
 
 On Thu, Jan 18, 2024 at 09:15:18AM -0700, Jerry James wrote:
> On Thu, Jan 18, 2024 at 4:50 AM Tomas Hrcka <thr...@redhat.com> wrote:
> > This is not a good idea. Because of a few packages that are not rebuilding 
> > we would disable the hook for every push the script does.
> 
> My thinking is that the hook is not useful for automated build scripts
> anyway, so disabling it doesn't hurt.  See my previous replies in this
> thread.

yeah... 

* Current setup with the hook:
  Bunch of fonts don't even get touched by the mass rebuild
  Some other packages with problems caught by the hook also are
completely missed.

* With --no-verify:
  Bunch of fonts do rebuild in the mass rebuild
  Some packages with problems get a mass rebuild commit at least and
attempted build. It might fail, but it's at least tried.

The mass rebuild is only doing a bump/rebuild. There's no reason it
should ever cause something that be caught by the hook, and if it did,
it would be better for it to do the commit anyhow and cause a failed
build. IMHO.

So, I think we should run with --no-verify.

kevin
--
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue
  
--
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to