Re: [Kaldi-developers] Windows Visual Studio build attempt and patches

[Michael F. <mic...@sr...>]

Sounds good. Thank you.

I will keep you posted with additional updates.

Mike

On 2/19/2015 12:58 PM, Daniel Povey wrote:
> I have committed fixes for these issues (I fixed the 
> comment-in-Makefile issue by changing the .pl script).  I hope the 
> perl script still works, I changed the newline style to UNIX.
> Let us know if you have further fixes.
> Dan
>
>
> On Thu, Feb 19, 2015 at 1:51 PM, Michael Frandsen 
> <mic...@sr... <mailto:mic...@sr...>> wrote:
>
>     Potential issues:
>     - Looks like openfst builds 32 bit (not 64) and uses VS 2010
>     - Kaldi will target 64 bit and uses VS 2012
>     (not sure if this is an issue yet)
>
>     Patches:
>
>     (1) The perl script that generates a VS 2012 project file from
>     Makefiles
>     tries to include a non-present file due to the perl script not
>     interpreting # as a comment. Patch:
>
>     $ svn diff src/matrix/Makefile
>     Index: src/matrix/Makefile
>     ===================================================================
>     --- src/matrix/Makefile (revision 4904)
>     +++ src/matrix/Makefile (working copy)
>     @@ -10,7 +10,8 @@
>
>       # you can uncomment matrix-lib-speed-test if you want to do the
>     speed
>     tests.
>
>     -TESTFILES = matrix-lib-test kaldi-gpsr-test #matrix-lib-speed-test
>     +#matrix-lib-speed-test
>     +TESTFILES = matrix-lib-test kaldi-gpsr-test
>
>       OBJFILES = kaldi-matrix.o kaldi-vector.o packed-matrix.o sp-matrix.o
>     tp-matrix.o \
>                  matrix-functions.o qr.o srfft.o kaldi-gpsr.o
>     compressed-matrix.o \
>
>
>     (2) generate_solution.pl <http://generate_solution.pl> has a typo.
>     Patch:
>
>     $ svn diff windows/generate_solution.pl <http://generate_solution.pl>
>     Index: windows/generate_solution.pl <http://generate_solution.pl>
>     ===================================================================
>     --- windows/generate_solution.pl <http://generate_solution.pl>    
>       (revision 4904)
>     +++ windows/generate_solution.pl <http://generate_solution.pl>    
>       (working copy)
>     @@ -394,7 +394,7 @@
>         </PropertyGroup>
>         <PropertyGroup
>     Condition=\"'\$(Configuration)|\$(Platform)'=='Release|Win32'\"
>     Label=\"Configuration\">
>           <ConfigurationType>" . $conftype . "</ConfigurationType>
>     -    <CharacterSet>Unicode</CharacterSeti>
>     +    <CharacterSet>Unicode</CharacterSet>
>           <PlatformToolset>v110</PlatformToolset>
>     <WholeProgramOptimization>true</WholeProgramOptimization>
>         </PropertyGroup>
>
>
>     Mike
>
>     ---
>     This email has been checked for viruses by Avast antivirus software.
>     http://www.avast.com
>
>
>     ------------------------------------------------------------------------------
>     Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
>     from Actuate! Instantly Supercharge Your Business Reports and
>     Dashboards
>     with Interactivity, Sharing, Native Excel Exports, App Integration
>     & more
>     Get technology previously reserved for billion-dollar
>     corporations, FREE
>     http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk
>     _______________________________________________
>     Kaldi-developers mailing list
>     Kal...@li...
>     <mailto:Kal...@li...>
>     https://lists.sourceforge.net/lists/listinfo/kaldi-developers
>
>



---
This email has been checked for viruses by Avast antivirus software.
http://www.avast.com