Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gforge - GForge Collaborative Development Environment https://bugzilla.redhat.com/show_bug.cgi?id=449135 kwizart@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kwizart@xxxxxxxxx ------- Additional Comments From kwizart@xxxxxxxxx 2008-07-29 11:33 EST ------- Just a quick look and few advices. 1/ While using cp (ex: cp cronjobs/cvs-cron/cvs.php %{buildroot}/%{gforge_dir}/cronjobs ) you may need to prevent timestamps changes by using cp -p (or install -pm 0644 ) 2/ -e 's/192\.168\.100\.100/127.0.0.1/' \ seems strange for a default install nobody know which sub-network will be used on the end-users system. Unfortunately that will probably remains a end-users post install command install 3/ # Set password for admin user (pw=admin) SITEADMIN_PASSWORD="21232f297a57a5a743894a0e4a801fc3" This seems scary at first sight... Does the patches bundled will be merged in the next gforge release ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review