[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[Pkg-octave-devel] Bug#299502: matwrap: stubs with dot in the name of one of their leading



package matwrap
tags 299502 upstream
forwarded 299502 holt@gholt.net
stop

The bug report below has been filled against the matwrap package in Debian.
The bug is real and is fixed by the following patch:

--- wrap_octave.pl-orig	2005-04-21 18:29:57.093231744 +0200
+++ wrap_octave.pl	2005-04-21 18:30:07.301006216 +0200
@@ -359,8 +359,8 @@
     die("wrap_octave: can't open output file $outfile--$!\n");
 
   $octave_output_file = $outfile; # Remember the name of the output file.
-  $octave_output_file =~ s/\..*$//; # Strip off the extension.
   $octave_output_file =~ s@.*/@@; # Strip off the directories.
+  $octave_output_file =~ s/\..*$//; # Strip off the extension.
 
   print OUTFILE "/*
  * This file was automatically generated for octave by wrap_octave on

Best regards,

-- 
Rafael


* Benoit Hamet <benoit.hamet@mist-technologies.com> [2005-03-14 15:48]:

> Package: matwrap
> Subject: matwrap: stubs with dot in the name of one of their leading
> directories implies no function name (at least octave stubs)
> Version: 0.57-5
> Severity: normal
> 
> *** Please type your report below this line ***
> Hello,
> 
> I'm used to put evry generated stuff (even .m) in a special directory
> typically, something like : ".build-architecture"
> 
> But when calling matwrap with the .build-architecture/test/test.m for
> example, the generated test_octave.cpp will get an empty
> DEFUN_DLD(,,...) in this case.
> 
> This is "easily" corrected by changing the order of the stripping
> process (first remove directory, then remove the . of extension
> filename) in matwrap_octave.pl line 362 and 363 should be switched to
> get the expected result.
> 
> I don't test it for tesla and matlab, but looking at them, they seems to
> not get the same line (but I don't see the stripping of dirs in tesla
> and the process seems different for matlab).
> 
> I can give a patch If you need it.
> 
> Thanks for your attention.
> 
> Regards.
> Benoit
> 
> -- System Information:
> Debian Release: 3.1
> Architecture: amd64 (x86_64)
> Kernel: Linux 2.6.8-10-amd64-k8
> Locale: LANG=fr_FR@euro, LC_CTYPE=fr_FR@euro (charmap=ISO-8859-15)
> 
> Versions of packages matwrap depends on:
> ii  perl                          5.8.4-6    Larry Wall's Practical
> Extraction
> 
> -- no debconf information
> 




Reply to: