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

[Daniel P. <dp...@gm...>]

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...>
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 has a typo. Patch:
>
> $ svn diff windows/generate_solution.pl
> Index: windows/generate_solution.pl
> ===================================================================
> --- windows/generate_solution.pl        (revision 4904)
> +++ windows/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...
> https://lists.sourceforge.net/lists/listinfo/kaldi-developers
>