[Bug 68902] Add pre-commit hook to check executable rights

2014-08-11 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=68902

Andre Klapper aklap...@wikimedia.org changed:

   What|Removed |Added

   Priority|Normal  |Lowest
   Severity|normal  |enhancement

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 68902] Add pre-commit hook to check executable rights

2014-08-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=68902

--- Comment #2 from db duplicate...@googlemail.com ---
There is a hook, which rejectes patch sets without valid Change-Id, the same is
needed here, maybe call it post-commit hook or server-commit hook, but that is
wat is needed to avoid having these files in gerrit. To avoid merge of this,
you can use jenkins, but not all repos having jenkins running.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 68902] Add pre-commit hook to check executable rights

2014-08-10 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=68902

--- Comment #3 from Chad H. innocentkil...@gmail.com ---
(In reply to db from comment #2)
 There is a hook, which rejectes patch sets without valid Change-Id, the same
 is needed here, maybe call it post-commit hook or server-commit hook, but
 that is wat is needed to avoid having these files in gerrit. To avoid merge
 of this, you can use jenkins, but not all repos having jenkins running.

That's not a hook. It's a piece of functionality in Gerrit. I have no
intentions of hacking Gerrit for this when Jenkins will do just fine. If
Jenkins isn't running on a repo then add it.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 68902] Add pre-commit hook to check executable rights

2014-07-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=68902

christ...@quelltextlich.at changed:

   What|Removed |Added

 CC||g...@wikimedia.org,
   ||has...@free.fr,
   ||krinklem...@gmail.com
  Component|Git/Gerrit  |Continuous integration

--- Comment #1 from christ...@quelltextlich.at ---
You can of course install your own pre-commit hook to do so, but git
does not allow to automatically install hooks.

But one could add such guards to Jenkins, if that's desired.
Hence, moving to the Continuous integration component.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 68902] Add pre-commit hook to check executable rights

2014-07-31 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=68902

Greg Grossmeier g...@wikimedia.org changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal
 CC||gla...@hallowelt.biz,
   ||hexm...@gmail.com,
   ||m...@everybody.org

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l