Closed Bug 407785 Opened 17 years ago Closed 17 years ago

Bootstrap::Util::CvsTag should log what it's going to do

Categories

(Release Engineering :: General, defect, P3)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

Details

Attachments

(2 files)

[checked in] make cvs tag log what it is going to do before it does it
820 bytes, patch
rhelmer
: review+
Details | Diff | Splinter Review
add newlines to prints
980 bytes, patch
rhelmer
: review+
Details | Diff | Splinter Review 
Most other things in Bootstrap log what they are about to. CvsCo(), any shell command that goes through Bootstrap::Step::Shell(), etc. CvsTag should do this too. This could mean moving it to Bootstrap::Step:: or just duplicating the log dumping function in the CvsTag() method.
Priority: -- → P3
Summary: Bootstrap::Util::CvsTag should what it's going to → Bootstrap::Util::CvsTag should log what it's going to
Attachment #293572 - Flags: review?(rhelmer) → review+
Summary: Bootstrap::Util::CvsTag should log what it's going to → Bootstrap::Util::CvsTag should log what it's going to do
Checking in Bootstrap/Util.pm;
/cvsroot/mozilla/tools/release/Bootstrap/Util.pm,v  <--  Util.pm
new revision: 1.12; previous revision: 1.11
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Forgot to add newlines to these statements.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

        Attachment #293760 -
        Flags: review?(rhelmer)

    
  

        Attachment #293572 -
        Attachment description: make cvs tag log what it is going to do before it does it → [checked in] make cvs tag log what it is going to do before it does it

        Attachment #293760 -
        Flags: review?(rhelmer) → review+

    
    

    
  
Checking in Bootstrap/Util.pm;
/cvsroot/mozilla/tools/release/Bootstrap/Util.pm,v  <--  Util.pm
new revision: 1.13; previous revision: 1.12
done


    
  
Status: REOPENED → RESOLVED
Closed: 17 years ago → 17 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: