Message44013
Logged In: YES
user_id=21627
I like the feature, but not the implementation:
- The option should not be called "record-rpm", as it does
not record
RPM files, but directories.
- RPM-specific code should be completely restricted to the
bdist_rpm
command. For the case of directories, you should create a
mechanism
to collect all directories, and then traverse this list to
create the
proper %dir directives in the spec file.
- I'm not sure that overloading get_outputs is a good idea.
Instead,
introducing get_output_dirs might be better.
|
|
Date |
User |
Action |
Args |
2007-08-23 15:27:43 | admin | link | issue755286 messages |
2007-08-23 15:27:43 | admin | create | |
|