[rt-devel] Confused!

[Ben Laurie]

My last report was wrong - the patch supplied _doesn't_ fix RT 1.0.7 - I
must've confused myself!

This one does (at the cost of making generic_templates writable by root,
which doesn't bother me much)...

diff -ur rt-1.0.7/Makefile rt-1.0.7-ben/Makefile
--- rt-1.0.7/Makefile	Thu Jan 18 08:18:59 2001
+++ rt-1.0.7-ben/Makefile	Sun May  6 15:24:28 2001
@@ -239,7 +239,7 @@
 fixperms:
 	#  Main, read-only directories.
 	chown -R root $(RT_PATH)
-	chgrp -R root $(RT_PATH)
+	chgrp -R wheel $(RT_PATH)
 	chmod -R a+r,a-w $(RT_PATH)
 	( cd $(RT_PATH) && find . -type d -exec chmod a+x {} \; )
 	#  Restricted and special access areas.
@@ -257,7 +257,8 @@
 	# libraries should be world readable
 	( cd $(RT_LIB_PATH) && find . -type d -exec chmod ugo+rx {} \; )
 	( cd $(RT_LIB_PATH) && find . -type f -exec chmod ugo+r {} \; )
-	
+	# generic templates need to be writable
+	chmod u+w $(RT_LIB_PATH)/generic_templates
 	#  Individual executable files.
 	chmod 0755 $(RT_PERL_MUX)
 	chown $(RTUSER) $(RT_WRAPPER)
diff -ur rt-1.0.7/lib/rt/database/admin.pm
rt-1.0.7-ben/lib/rt/database/admin.pm
--- rt-1.0.7/lib/rt/database/admin.pm	Wed Jul  7 19:37:10 1999
+++ rt-1.0.7-ben/lib/rt/database/admin.pm	Sun May  6 15:24:50 2001
@@ -129,7 +129,7 @@
       $query_string="INSERT INTO queues (queue_id, mail_alias,
m_owner_trans,  m_members_trans, m_user_trans, m_user_create,
m_members_corresp,m_members_comment, allow_user_create, default_prio,
default_final_prio) VALUES ($queue_id, $mail_alias, $in_m_owner_trans,
$in_m_members_trans, $in_m_user_trans, $in_m_user_create,
$in_m_members_correspond, $in_m_members_comment, $in_allow_user_create,
$in_default_prio, $in_default_final_prio)";
       $dbh->Query($query_string) or return (0, "[add_modify_queue]
Query had some problem: $Mysql::db_errstr\n$query_string is query\n");
       $< = $>;			#set real to effective uid
-      system("cp", "-rp",
"$rt_dir/lib/generic_templates","$template_dir/queues/$in_queue_id");
+      system("cp", "-rp",
"$rt_dir/lib/generic_templates/","$template_dir/queues/$in_queue_id");
       &rt::load_queue_conf();
       return(1,"Queue $in_queue_id sucessfully created.");   
     }

Cheers,

Ben.

--
http://www.apache-ssl.org/ben.html

"There is no limit to what a man can do or how far he can go if he
doesn't mind who gets the credit." - Robert Woodruff