[haiku-commits] Re: haiku: hrev51634 - build/jam/repositories/HaikuPorts

  • From: "Adrien Destugues" <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 29 Nov 2017 14:32:04 +0000

29 novembre 2017 15:23 "kallisti5" <kallisti5@xxxxxxxxxxx> a √©crit:

Sure it is! My concern is once these files are incorrectly changed,
everyone's build of Haiku brakes.

waddlesplash disabled the repository generation + check git hooks on push
since they don't make sense anymore.

I guess a better solution could be making a git hook that simply checks
the sha256hash of those repository URL's and validates the URL exists
before accepting commits to those files?

aka:
1) file in that path changed.
2) sha256sum is xxxyyy
3) url per repository file is https://blah.com
4) does https://blah.com/xxxyyy/repo exist?

I considered this as well, but eventually agreed that like any other build 
failure, this is caught by buildbots and by attempting to build locally, so it 
shouldn't be a problem (not more than usual).

The proper fix is not a specific git hook, but completing our switch to Gerrit 
so change requests submitted there are tested by a buildbot, which would notice 
such problems.

-- 
Adrien.

Other related posts: