[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug-gnu-arch] [bug #5690] Add a precommit hook

From: nobody
Subject: [Bug-gnu-arch] [bug #5690] Add a precommit hook
Date: Tue, 14 Oct 2003 17:10:33 -0400
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030930 Galeon/1.3.9

=================== BUG #5690: LATEST MODIFICATIONS ==================

Changes by: Robert Collins <address@hidden>
Date: Tue 10/14/2003 at 21:10 (GMT)

            What     | Removed                   | Added
                  CC |                           | robertc

=================== BUG #5690: FULL BUG SNAPSHOT ===================

Submitted by: mega-nerd               Project: GNU arch -- a revision control 
Submitted on: Thu 10/02/2003 at 10:55
Category:  tla                        Severity:  5 - Major                  
Bug Group:  small feature idea        Resolution:  None                     
Status:  Open                         Release:  tla-1.1pre5                 
Fixed Release:                        Merge Request?:  None                 
Your Archive Name:                    Your Archive Location:                
Assigned to:  None                    

Summary:  Add a precommit hook

Original Submission:  There is currently a hook in the commit function which 

at the end of the commit process. It would also be useful 

to have a hook run at the start of the commit process and 

be able to abort the commit process by returning a non-zero

error code from the hook script.

An example of what can be done with this is to do a make in 

the the hook script and have the script return make's error

code. This means that a compile will be done during the 

commit hook and if the code doesn't build it won't commit

and broken code doesn't get into the archive.

Obviously if someone wants to bypass this process they can.

This is meant to prevent accidental commits of broken code.

I have included a diff which seems to work for me, but I'm

not totally sure that its correct. I have also mentioned 

the previous version of this patch on the mailing list.

No Followups Have Been Posted

CC List

CC Address                          | Comment
robertc                             | 

File Attachments

Date: Thu 10/02/2003 at 10:55  Name: tla-1.1pre5-precommit.diff  Size: 1KB   
By: mega-nerd
A patch that seems to work.

For detailed info, follow this link:

  Message sent via/by Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]