[Python-bugs-list] [ python-Bugs-466200 ] ability to specify a 'verify' script

noreply@sourceforge.net noreply@sourceforge.net
Wed, 06 Nov 2002 10:47:11 -0800


Bugs item #466200, was opened at 2001-09-28 21:05
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=466200&group_id=5470

Category: Distutils
Group: Feature Request
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Jon Nelson (jnelson)
Assigned to: A.M. Kuchling (akuchling)
Summary: ability to specify a 'verify' script

Initial Comment:
The following patch adds the ability to specify a
'verify' script (as used by the %verify).

Treatment is exactly the same as with post_install,
etc...


--- bdist_rpm.py        Wed Sep 12 11:42:17 2001
+++ /home/agamemnon/jnelson/bdist_rpm.py        Fri Sep
28 16:09:18 2001
@@ -131,6 +131,7 @@
         self.post_install = None
         self.pre_uninstall = None
         self.post_uninstall = None
+        self.verifyscript = None
         self.prep = None
         self.provides = None
         self.requires = None
@@ -210,6 +211,7 @@
         self.ensure_filename('post_install')
         self.ensure_filename('pre_uninstall')
         self.ensure_filename('post_uninstall')
+        self.ensure_filename('verifyscript')
 
         # XXX don't forget we punted on summaries and
descriptions -- they
         # should be handled here eventually!
@@ -423,6 +425,7 @@
             ('post', 'post_install', None),
             ('preun', 'pre_uninstall', None),
             ('postun', 'post_uninstall', None),
+            ('verify', 'verifyscript', None),
         ]
 
         for (rpm_opt, attr, default) in
script_options:


----------------------------------------------------------------------

>Comment By: Gustavo Niemeyer (niemeyer)
Date: 2002-11-06 18:47

Message:
Logged In: YES 
user_id=7887

Applied in the following CVS revisions:   
 
Lib/distutils/command/bdist_rpm.py: 1.33->1.34 
Misc/NEWS: 1.513->1.514 
   
I've done a few changes to your original patch:   
   
- Named the option verify_script, for conformance with other options   
(clean_script, etc).   
   
-  The rpm section is named "%verifyscript", not "%verify". I've reflected  
that in the script_options tuple.  
  
Thank you!  

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=105470&aid=466200&group_id=5470