[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-devel
Subject:    am_edit and uic
From:       Alessandro Praduroux <pradu () thekompany ! com>
Date:       2000-09-14 10:05:28
[Download RAW message or body]

Hi all,

Here is the problem:
I design my widgets with designer, then derive a class from the generated one.
I use only my derived class in my app, so the only place where the header 
generated header file is included is the derived class header file.

Since am_edit only checks for $cppExt when generating the dependence, my 
Makefile will not contain the dependence between the derived .h file and the 
generated .h file.

The attached patch fixes this.

Can someone test and apply? (I have no CVS write access)

-- 
Pradu
-----------------------------------------
Alessandro Praduroux
pradu@thekompany.com
http://www.thekompany.com


["am_edit_uic.diff" (text/plain)]

Index: am_edit
===================================================================
RCS file: /public/kdb/admin/am_edit,v
retrieving revision 1.3
diff -u -3 -p -r1.3 am_edit
--- am_edit	2000/09/14 08:24:06	1.3
+++ am_edit	2000/09/14 10:17:48
@@ -921,7 +921,7 @@ sub tag_UIFILES ()
 {
   my @psources = split(/[\034\s]+/, $sources{$program});
   my $dep_lines = "";
-  my @cppFiles = ();
+  my @depFiles = ();
 
   foreach $source (@psources) {
 
@@ -948,12 +948,12 @@ sub tag_UIFILES ()
 	$dep_lines .= "$source.moc: $source.h\n";
 	$dep_lines .= "\t\$(MOC) $source.h -o $source.moc\n";
 
-	@cppFiles = `grep -l "^#include *.$source\.h." $cppExt 2> /dev/null`;
-        foreach $file (@cppFiles) {
+	@depFiles = `grep -l "^#include *.$source\.h." $cppExt $hExt 2> /dev/null`;
+	foreach $file (@depFiles) {
 	  chomp ($file);
 	  next unless $file;
 	  $dep_lines .= "\$(srcdir)/$file: $source.h\n";
-	}
+        }
 	
 	$alluis .= "$source ";
 	$ui_mocs .= " $source.moc";

>> Visit http://master.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic