iniwf / webm

Automatically exported from code.google.com/p/webm
0 stars 0 forks source link

make dist installs only the Release library to the dist dir for MSVC targets #889

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Not sure if this is the intent or not, but unexpected from my perspective.

Might end up wont-fix due to scripted project generation complexity.

Original issue reported on code.google.com by tomfine...@google.com on 3 Dec 2014 at 9:36

GoogleCodeExporter commented 9 years ago
Could install everything -- only a variable controls the target selection I 
believe -- though that would be a bit inconsistent with every other platform.

Original comment by jz...@google.com on 3 Dec 2014 at 10:27

GoogleCodeExporter commented 9 years ago
does --enable-debug-libs work with either the install or dist targets with msvs?

Original comment by jz...@google.com on 4 Dec 2014 at 1:58

GoogleCodeExporter commented 9 years ago

Original comment by renganat...@google.com on 4 Dec 2014 at 10:55

GoogleCodeExporter commented 9 years ago
Booo @pangu :)

No joy on --enable-debug:

$ rm -rf ./* \
  && ../configure --target=x86_64-win64-vs12 --enable-debug --enable-libyuv \
    --enable-static-msvcrt --disable-examples --disable-docs --disable-unit-tests \
  && make && DIST_DIR=_dist make

[...]

Build succeeded.
  0 Warning(s)
  0 Error(s)
Time Elapsed 00:00:32.84
    [INSTALL] _dist/include/vpx/vp8.h
    [INSTALL] _dist/include/vpx/vp8cx.h
    [INSTALL] _dist/include/vpx/vp8dx.h
    [INSTALL] _dist/include/vpx/vpx_codec.h
    [INSTALL] _dist/include/vpx/vpx_frame_buffer.h
    [INSTALL] _dist/include/vpx/vpx_image.h
    [INSTALL] _dist/include/vpx/vpx_integer.h
    [INSTALL] _dist/include/vpx/vpx_decoder.h
    [INSTALL] _dist/include/vpx/vpx_encoder.h
    [INSTALL] _dist/lib/x64/vpxmt.lib
make[1]: Nothing to be done for `dist'.
    [MD5SUM] _dist

And:

$ ls -1 _dist/lib/x64/
vpxmt.lib

Original comment by tomfine...@google.com on 4 Dec 2014 at 11:58

GoogleCodeExporter commented 9 years ago
it's spelled '--enable-debug-libs'. --enable-debug adds debugging info

Original comment by jz...@google.com on 5 Dec 2014 at 12:06

GoogleCodeExporter commented 9 years ago
Oops, my fault. Works with --enable-debug-libs. 

Original comment by tomfine...@google.com on 5 Dec 2014 at 12:16