#6448 zip outputs too much to logs

v1.0.0
closed
sf-1 (616)
General
2015-08-20
2013-07-10
No

The zip command used in code snapshots sends a lot of logs to stdout. Some is standard like:

adding: msieve-code-912-trunk/mpqs/poly.c (deflated 69%)

And some are warnings (so maybe stderr):

zip warning: could not open for reading: game-editor-code-501-trunk/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/android-project/jni/ge/gameEngine/EiC/test/EiCtests/testtypedef.c.def

We don't want to lose this altogether (e.g. if the zip fails, we want to see the stdout & stderr) but in general when the zip works we can remove the logging.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-07-15
    • Size: --> 1
     
    • status: open --> in-progress
    • assigned_to: Tim Van Steenburgh
     
    • status: in-progress --> code-review
     
  • forge:tv/6448

     
  • Dave Brondsema

    Dave Brondsema - 2013-07-19
    • QA: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2013-07-19
    • status: code-review --> closed
     

Log in to post a comment.