View Issue Details

IDProjectCategoryView StatusLast Update
0002328OpenFOAMPatchpublic2016-11-13 22:04
Reporteralexeym Assigned Tohenry  
PrioritynormalSeverityfeatureReproducibilityalways
Status resolvedResolutionfixed 
PlatformGNU/LinuxOSFedoraOS Version24
Product Versiondev 
Fixed in Versiondev 
Summary0002328: Silent rules for wmake
DescriptionCompilation logs in general contain lots of information in general rather useless to user: full compiler invocation commands with all flags. Patch introduces WM_SILENT_RULES environment variable, so build log files instead of

$ wmake
$HOME/OpenFOAM/OpenFOAM-dev/applications/solvers/incompressible/icoFoam
Making dependency list for source file icoFoam.C
g++ -std=c++11 -m64 -Dlinux64 -DWM_ARCH_OPTION=64 -DWM_DP -DWM_LABEL_SIZE=32 -Wall -Wextra -Wold-style-cast -Wnon-virtual-dtor -Wno-unused-parameter -Wno-invalid-offsetof -O3 -DNoRepository -ftemplate-depth-100 -I$HOME/OpenFOAM/OpenFOAM-dev/src/finiteVolume/lnInclude -I$HOME/OpenFOAM/OpenFOAM-dev/src/meshTools/lnInclude -IlnInclude -I. -I$HOME/OpenFOAM/OpenFOAM-dev/src/OpenFOAM/lnInclude -I$HOME/OpenFOAM/OpenFOAM-dev/src/OSspecific/POSIX/lnInclude -fPIC -c icoFoam.C -o $HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/applications/solvers/incompressible/icoFoam/icoFoam.o
g++ -std=c++11 -m64 -Dlinux64 -DWM_ARCH_OPTION=64 -DWM_DP -DWM_LABEL_SIZE=32 -Wall -Wextra -Wold-style-cast -Wnon-virtual-dtor -Wno-unused-parameter -Wno-invalid-offsetof -O3 -DNoRepository -ftemplate-depth-100 -I$HOME/OpenFOAM/OpenFOAM-dev/src/finiteVolume/lnInclude -I$HOME/OpenFOAM/OpenFOAM-dev/src/meshTools/lnInclude -IlnInclude -I. -I$HOME/OpenFOAM/OpenFOAM-dev/src/OpenFOAM/lnInclude -I$HOME/OpenFOAM/OpenFOAM-dev/src/OSspecific/POSIX/lnInclude -fPIC -Xlinker --add-needed -Xlinker --no-as-needed $HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/applications/solvers/incompressible/icoFoam/icoFoam.o -L$HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib \
    -lfiniteVolume -lmeshTools -lOpenFOAM -ldl \
     -lm -o $HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/bin/icoFoam

user gets

$ wmake
$HOME/OpenFOAM/OpenFOAM-dev/applications/solvers/incompressible/icoFoam
   DEP icoFoam.C
    CC icoFoam.C
    LD icoFoam
Additional InformationTo use the feature, set WM_SILENT_RULES to non-empty value.
Tagswmake

Activities

alexeym

2016-11-11 08:40

reporter  

0003-Silent-rules.patch (12,153 bytes)   

henry

2016-11-11 09:58

manager   ~0007139

I think this functionality would be more useful if wmake/Allwmake have a command-line option to set WM_SILENT_RULES.

alexeym

2016-11-11 21:49

reporter   ~0007162

Added

1. -silent-rules option to wmake.
2. -silentRules option to AllwmakeParseArguments.

and kept ability to enable it globally by setting WM_SILENT_RULES environment variable.

Tried to guess option construction rules and it seems that for wmake it is -an-option, for AllwmakeParseArguments it is -anOption.
Silent-rules.patch (14,639 bytes)   

henry

2016-11-11 22:47

manager   ~0007163

Allwmake command-line options are now handled via wmake:

./Allwmake -help

Usage: Allwmake [OPTIONS]

Executing Allwmake is equivalent to

   wmake -all [OPTIONS]

Usage: wmake [OPTION] [dir]
       wmake [OPTION] target [dir [MakeDir]]

options:
  -s | -silent 'silent' mode (does not echo commands)
  -a | -all wmake all sub-directories, running Allwmake if present
  -q | -queue wmakeQueue all sub-directories, running Allwmake if present
  -k or -non-stop Compile without stopping when errors occur
  -j Compile using all local cores/hyperthreads
  -jN or -j N Compile using N cores/hyperthreads
  -no-scheduler Compile without wmakeScheduler
  -update Update lnInclude directories, dep files, remove deprecated
                    files and directories
  -h | -help Print the usage


A general, easy-to-use make system for multi-platform development
with support for local and network parallel compilation.

The 'target' is a Makefile target:
  e.g., platforms/linux64GccDPOpt/.../fvMesh.o

or a special target:
  all wmake all sub-directories, running Allwmake if present
  queue wmakeQueue all sub-directories, running Allwmake if present
  exe Compile statically linked executable
  lib Compile statically linked archive lib (.a)
  libo Compile statically linked lib (.o)
  libso Compile dynamically linked lib (.so)
  dep Compile lnInclude and dependencies only
  updatedep Compile dependencies only (in case of broken dependencies)
  objects Compile but not link

alexeym

2016-11-12 09:49

reporter   ~0007165

Removed -silentRules in AllwmakeParseArguments. So now it is either -silent-rules in wmake/Allwmake invocation or export WM_SILENT_RULES=Y to enable it globally.

henry

2016-11-12 12:01

manager   ~0007166

I am working on this at the moment and wondering if both the -s | -silent and -silent-rules are needed, my feeling is that either -silent-rules should replace the current -silent option or they should be combined.

henry

2016-11-12 13:41

manager   ~0007167

I am not happy with the amount of clutter and code duplication this change introduces, for example

 
 $(OBJECTS_DIR)/%.dep : %
        @$(WM_SCRIPTS)/makeTargetDir $@
+ifneq ("$(WM_SILENT_RULES)","")
        @$(WMAKE_BIN)/wmkdep -I$(*D) $(LIB_HEADER_DIRS) $< | \
                sed -e 's,^$(WM_PROJECT_DIR)/,$$(WM_PROJECT_DIR)/,' \
                        -e 's,^$(WM_THIRD_PARTY_DIR)/,$$(WM_THIRD_PARTY_DIR)/,' > $@
+else
+ @$(WMAKE_BIN)/wmkdep -v -I$(*D) $(LIB_HEADER_DIRS) $< | \
+ sed -e 's,^$(WM_PROJECT_DIR)/,$$(WM_PROJECT_DIR)/,' \
+ -e 's,^$(WM_THIRD_PARTY_DIR)/,$$(WM_THIRD_PARTY_DIR)/,' > $@
+endif

where the only difference between the calls to wmkdep appears to be the '-v' option. Would it be better to define this additional argument according to the setting of WM_SILENT_RULES?

Even the duplication in

 define DEFINE_TRANSFORM
 $(OBJECTS_DIR)/%.o : %$1
+ifneq ("$(WM_SILENT_RULES)","")
+ @echo " CC $(value <F)"
+ @$$($(subst .,,$(1))too)
+else
        $$($(subst .,,$(1))too)
+endif
 endef
 
could be avoided by setting a prefix variable which is either '@' or '' depending on the value of WM_SILENT_RULES.

henry

2016-11-12 16:52

manager   ~0007168

This basically hangs together and avoids the code duplication by switching the definitions of E and the function SILENT_MESSAGE. I think it should be renamed QUIET rather than SILENT because it isn't actually silent.

#-------------------------------*- makefile -*---------------------------------

ifeq ("$(WM_SILENT_RULES)","")
    E=

    define SILENT_MESSAGE
        @echo $1
    endef

    SILENT_OP=-s
else
    E=@

    define SILENT_MESSAGE
        @echo $1
    endef

    SILENT_OP=
endif

define DEFINE_TRANSFORM
$(OBJECTS_DIR)/%.o : %$1
    $(call SILENT_MESSAGE, " Compile $(value <F)")
    $E $$($(subst .,,$(1))too)
endef

$(foreach s,$(SUFFIXES),$(eval $(call DEFINE_TRANSFORM,$(s))))

$(OBJECTS_DIR)/%.dep : %
    @$(WM_SCRIPTS)/makeTargetDir $@
    $E $(WMAKE_BIN)/wmkdep $(SILENT_OP) -I$(*D) $(LIB_HEADER_DIRS) $< | \
        sed -e 's,^$(WM_PROJECT_DIR)/,$$(WM_PROJECT_DIR)/,' \
            -e 's,^$(WM_THIRD_PARTY_DIR)/,$$(WM_THIRD_PARTY_DIR)/,' > $@

#------------------------------------------------------------------------------

henry

2016-11-12 18:41

manager   ~0007169

I have updated most of the wmake files without all the mess and code duplication and can now build with the '-s' option but it interacts badly with the '-q' option which will require more digging, careful testing and tweaking.

Have you tried your version with the '-q' option?

alexeym

2016-11-12 21:40

reporter   ~0007170

The patch originated from 4.x, so it (just like me) does not know anything about -q option. I have checked compatibility and it is awful due to non-propagation of command line options. I will update patch ASAP (not quite sure I will be able to integrate your modifications).

henry

2016-11-12 22:13

manager   ~0007171

I have updated wmakeCollect to handle WM_QUIET and now the behavior is reasonable although there are quite a few other messages generated by wmake which could do with the same treatment. I have changed all of the wmake/rules/General files to use the 'E' and 'QUIET_MESSAGE' macros to avoid the proliferation of conditional statements and the result is much cleaner. I have committed all of the changes so far:

OpenFOAM-dev commit a0094dee381c838314a17bca95c842d4253dbf17

I assumed you were running OpenFOAM-dev given that you opened this patch request for 'dev'. I recommend using the '-q' option which speeds-up compilation on a multi-core machine significantly and is proving very reliable.

alexeym

2016-11-13 09:14

reporter   ~0007172

What I have noticed so far:

1. wmake/src/Makefile: extra $E in front of $(cc) in wmkdep rule. It breaks "make WM_QUIET=1".
2. Is too suffix to be kept in DEFINE_TRANSFORM's QUIET_MESSAGE argument? Or it could be just

$(call QUIET_MESSAGE, " $(subst .,,$(1)): $(value <F)")

3. Quiet additions in rules/General/yacc are missing, guess, in my patch they were also absent.

4. Alignment: I would like to propose align : symbol by longest command (btyacc++ so far), so the output is like:

daphne:renumber$ ./Allwmake -s
$HOME/OpenFOAM/OpenFOAM-dev/src/renumber/renumberMethods
      ln: ./lnInclude
  wmkdep: OppositeFaceCellWaveName.C
  wmkdep: springRenumber.C
  wmkdep: structuredRenumber.C
  wmkdep: randomRenumber.C
  wmkdep: CuthillMcKeeRenumber.C
  wmkdep: manualRenumber.C
  wmkdep: renumberMethod.C
       C: manualRenumber.C
       C: renumberMethod.C
       C: CuthillMcKeeRenumber.C
       C: randomRenumber.C
       C: springRenumber.C
       C: structuredRenumber.C
       C: OppositeFaceCellWaveName.C
      ld: librenumberMethods.so
->> $HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so

henry

2016-11-13 09:52

manager   ~0007173

1. I haven't tested

2. The conversion rule is .*too because it converts to '.o'

3. OK, I will make this change.

4. Your proposed alignment looks odd; what is the advantage in having the 'C' align with the 'k' of wmkdep?

alexeym

2016-11-13 10:07

reporter   ~0007174

2. Then wmkdep label should be Ctodep. It is a matter of preferences having Ctoo or just C, so let it be Ctoo.

4. Variable width fonts break alignment in the note, with fixed width fonts output looks more aligned. See attached build.log.
build.log (689 bytes)   

henry

2016-11-13 10:21

manager   ~0007175

2. wmkdep is a conversion command whereas for the conversion of .C to .o the conversion command is actually either g++, clang++, icpc etc, which are wrapped into the Ctoo. It might be possible to work out which compiler is actually being used in the Ctoo macro and print that instead.

Note that wmake is not C++ specific and other langauage files can be converted into .o files so we should not hard-code the message for C++.

4. I don't see this as better than simple left justification, it would be better to use a 2-column format with left justification for the command space then the ':' in a column and the source file. This should not be done with spaces because it would require ALL of the message strings to be changed if a new conversion command is added which longer than the current commands.

alexeym

2016-11-13 11:02

reporter   ~0007176

wmake/rules/General/bison: missing closing " in QUIET_MESSAGE call

4. Initial idea behind silent rules is to reduce cognitive load while looking at build log files. The same reasoning is behind two-column output: first column is WHAT we are doing, second column is WHERE we are doing it. One can use left alignment in both columns and separate them with tabs; or it is possible to use printf command to right-align left column. Former solution needs additional variable: left column width.

henry

2016-11-13 11:15

manager   ~0007177

I found the problem in bison and fixed it while simplifying the QUIET_MESSAGE function

4. I don't have a problem with either left-justified one-column or two-column format but the two-column format would take additional effort.

henry

2016-11-13 11:22

manager   ~0007178

I have pushed the simplified QUIET_MESSAGE function:

commit 2b3f1d6a41b6cbb84d918bfefa70315b9d85dfa4

alexeym

2016-11-13 15:26

reporter   ~0007182

With simplified QUIET_MESSAGE function alignment implementation is rather simple. See my attempt in attachment.

-q flag's behavoiur right now is rather strange:

daphne:renumber$ ./Allwmake -q -s
wmakeLnIncludeAll: starting wmakeLnInclude processes on 4 cores
    searching . for 'Make' directories
        ln: ./SloanRenumber/lnInclude
        ln: ./zoltanRenumber/lnInclude
        ln: ./renumberMethods/lnInclude
$HOME/OpenFOAM/OpenFOAM-dev/src/renumber/renumberMethods
    wmkdep: OppositeFaceCellWaveName.C
    wmkdep: structuredRenumber.C
    wmkdep: randomRenumber.C
    wmkdep: springRenumber.C
    wmkdep: CuthillMcKeeRenumber.C
    wmkdep: manualRenumber.C
    wmkdep: renumberMethod.C
      Ctoo: renumberMethod.C
      Ctoo: manualRenumber.C
      Ctoo: randomRenumber.C
      Ctoo: CuthillMcKeeRenumber.C
      Ctoo: springRenumber.C
      Ctoo: structuredRenumber.C
      Ctoo: OppositeFaceCellWaveName.C

Skipping zoltanRenumber


    compiling: renumberMethod.C
    compiling: randomRenumber.C
    compiling: manualRenumber.C
    compiling: CuthillMcKeeRenumber.C
    compiling: structuredRenumber.C
    compiling: springRenumber.C
    compiling: OppositeFaceCellWaveName.C
/home/alexey/OpenFOAM/OpenFOAM-dev/src/renumber/renumberMethods
        ld: librenumberMethods.so
->> $HOME/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so

without -s flag files are also compiled twice.
0001-Colon-alignment.patch (4,428 bytes)   

henry

2016-11-13 15:39

manager   ~0007185

It is not compiling twice, it first generates a complete Makefile for all the object file compilations by running wmake echo'ing the commands into files which are then collected and the Makefile executed with make. For this second (real compilation) stage it is diffult to get hold of the actual compilation command so I put in "compiling:" instead.

You will notice that the complete build is significantly quicker than without the '-q'.

henry

2016-11-13 15:44

manager   ~0007186

This adds a lot of clutter to wmkdep duplicating in C what is also written in the Makefile. It would probably be better if the messaging from wmkdep.l is removed and the appropriate messages generated in the Makefile.

henry

2016-11-13 15:49

manager   ~0007187

Is the log in your last post generated with the new alignment function?
This alignment looks the same as your original format which does not left-justify the commands but aligns the ':'s which looks awful. My preference would be to either left-justify the commands as is currently done in OpenFOAM-dev or left-justify the commands AND align the ':'s.

henry

2016-11-13 18:14

manager   ~0007192

Cleaned-up the output from the Allwmake scripts:

commit 48e3fb78052699a00f6d1434a8b37c4e4c0d6ddc

alexeym

2016-11-13 18:56

reporter   ~0007193

Question of alignment:

daphne:renumber$ ./Allwmake -s
/home/alexey/OpenFOAM/OpenFOAM-dev/src/renumber/renumberMethods
ln        : ./lnInclude
wmkdep    : OppositeFaceCellWaveName.C
wmkdep    : structuredRenumber.C
wmkdep    : springRenumber.C
wmkdep    : randomRenumber.C
wmkdep    : CuthillMcKeeRenumber.C
wmkdep    : manualRenumber.C
wmkdep    : renumberMethod.C
Ctoo      : renumberMethod.C
Ctoo      : manualRenumber.C
Ctoo      : CuthillMcKeeRenumber.C
Ctoo      : randomRenumber.C
Ctoo      : springRenumber.C
Ctoo      : structuredRenumber.C
Ctoo      : OppositeFaceCellWaveName.C
ld        : librenumberMethods.so
->> /home/alexey/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so

Skipping zoltanRenumber


and

daphne:renumber$ ./Allwmake -s
/home/alexey/OpenFOAM/OpenFOAM-dev/src/renumber/renumberMethods
        ln: ./lnInclude
    wmkdep: OppositeFaceCellWaveName.C
    wmkdep: structuredRenumber.C
    wmkdep: springRenumber.C
    wmkdep: randomRenumber.C
    wmkdep: CuthillMcKeeRenumber.C
    wmkdep: manualRenumber.C
    wmkdep: renumberMethod.C
      Ctoo: renumberMethod.C
      Ctoo: CuthillMcKeeRenumber.C
      Ctoo: manualRenumber.C
      Ctoo: randomRenumber.C
      Ctoo: springRenumber.C
      Ctoo: structuredRenumber.C
      Ctoo: OppositeFaceCellWaveName.C
        ld: librenumberMethods.so
->> /home/alexey/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so

Skipping zoltanRenumber


with -q flag:

daphne:renumber$ ./Allwmake -s -q
Allwmake src/renumber
wmakeLnIncludeAll: starting wmakeLnInclude processes on 4 cores
    searching . for 'Make' directories
        ln: ./SloanRenumber/lnInclude
        ln: ./zoltanRenumber/lnInclude
        ln: ./renumberMethods/lnInclude
wmake objects .
Allwmake objects src/renumber
    skipping zoltanRenumber
wmake objects renumberMethods
    wmkdep: structuredRenumber.C
    wmkdep: OppositeFaceCellWaveName.C
    wmkdep: springRenumber.C
    wmkdep: randomRenumber.C
    wmkdep: CuthillMcKeeRenumber.C
    wmkdep: manualRenumber.C
    wmkdep: renumberMethod.C
      Ctoo: manualRenumber.C
      Ctoo: renumberMethod.C
      Ctoo: CuthillMcKeeRenumber.C
      Ctoo: randomRenumber.C
      Ctoo: structuredRenumber.C
      Ctoo: springRenumber.C
      Ctoo: OppositeFaceCellWaveName.C
 compiling: CuthillMcKeeRenumber.C
 compiling: renumberMethod.C
 compiling: manualRenumber.C
 compiling: randomRenumber.C
 compiling: structuredRenumber.C
 compiling: springRenumber.C
 compiling: OppositeFaceCellWaveName.C
Allwmake src/renumber
    skipping zoltanRenumber
wmake renumberMethods
        ld: librenumberMethods.so
    ->> /home/alexey/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so

henry

2016-11-13 20:13

manager   ~0007194

My view is at least the commands should be indented and left justified as they are currently, none of the options above have this.

My preference is for the current arrangement which I think has the simplest implementation, least clutter on the screen and easiest to propagate consistently through the rest of the build scripts. My second choice would be for the first of your options above but with the commands indented.

alexeym

2016-11-13 21:50

reporter   ~0007197

Well, left align, indent (initially ->> has 0 indentation to separate work units ->> was also selected cause initially all compilation/transformation indents was 6 (4 spaces + CC, RANLIB without indent) symbols)

daphne:renumber$ ./Allwmake -s -q
Allwmake src/renumber
wmakeLnIncludeAll: starting wmakeLnInclude processes on 4 cores
    searching . for 'Make' directories
    ln: ./SloanRenumber/lnInclude
    ln: ./renumberMethods/lnInclude
    ln: ./zoltanRenumber/lnInclude
wmake objects .
Allwmake objects src/renumber
    skipping zoltanRenumber
wmake objects renumberMethods
    wmkdep: structuredRenumber.C
    wmkdep: OppositeFaceCellWaveName.C
    wmkdep: springRenumber.C
    wmkdep: randomRenumber.C
    wmkdep: CuthillMcKeeRenumber.C
    wmkdep: manualRenumber.C
    wmkdep: renumberMethod.C
    Ctoo: renumberMethod.C
    Ctoo: manualRenumber.C
    Ctoo: randomRenumber.C
    Ctoo: CuthillMcKeeRenumber.C
    Ctoo: springRenumber.C
    Ctoo: structuredRenumber.C
    Ctoo: OppositeFaceCellWaveName.C
    compiling: renumberMethod.C
    compiling: manualRenumber.C
    compiling: randomRenumber.C
    compiling: CuthillMcKeeRenumber.C
    compiling: springRenumber.C
    compiling: OppositeFaceCellWaveName.C
    compiling: structuredRenumber.C
Allwmake src/renumber
    skipping zoltanRenumber
wmake renumberMethods
    ld: librenumberMethods.so
    ->> /home/alexey/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so


right align:

daphne:renumber$ ./Allwmake -s -q
Allwmake src/renumber
wmakeLnIncludeAll: starting wmakeLnInclude processes on 4 cores
    searching . for 'Make' directories
       ln: ./SloanRenumber/lnInclude
       ln: ./zoltanRenumber/lnInclude
       ln: ./renumberMethods/lnInclude
wmake objects .
Allwmake objects src/renumber
    skipping zoltanRenumber
wmake objects renumberMethods
   wmkdep: OppositeFaceCellWaveName.C
   wmkdep: structuredRenumber.C
   wmkdep: springRenumber.C
   wmkdep: randomRenumber.C
   wmkdep: CuthillMcKeeRenumber.C
   wmkdep: manualRenumber.C
   wmkdep: renumberMethod.C
     Ctoo: renumberMethod.C
     Ctoo: manualRenumber.C
     Ctoo: CuthillMcKeeRenumber.C
     Ctoo: randomRenumber.C
     Ctoo: springRenumber.C
     Ctoo: structuredRenumber.C
     Ctoo: OppositeFaceCellWaveName.C
compiling: renumberMethod.C
compiling: manualRenumber.C
compiling: CuthillMcKeeRenumber.C
compiling: randomRenumber.C
compiling: structuredRenumber.C
compiling: springRenumber.C
compiling: OppositeFaceCellWaveName.C
Allwmake src/renumber
    skipping zoltanRenumber
wmake renumberMethods
       ld: librenumberMethods.so
->> /home/alexey/OpenFOAM/OpenFOAM-dev/platforms/linux64GccDPInt32Opt/lib/librenumberMethods.so


Since our alignment and output preferences are quite different and I need to maintain own patch set anyway (for this, for Scotch/METIS, for other things), I think bug report can be closed, commit mention edited-out.

Issue History

Date Modified Username Field Change
2016-11-11 08:40 alexeym New Issue
2016-11-11 08:40 alexeym File Added: 0003-Silent-rules.patch
2016-11-11 08:40 alexeym Tag Attached: wmake
2016-11-11 09:58 henry Note Added: 0007139
2016-11-11 21:49 alexeym File Added: Silent-rules.patch
2016-11-11 21:49 alexeym Note Added: 0007162
2016-11-11 22:47 henry Note Added: 0007163
2016-11-12 09:49 alexeym File Added: 0001-Silent-rules-for-wmake.patch
2016-11-12 09:49 alexeym Note Added: 0007165
2016-11-12 12:01 henry Note Added: 0007166
2016-11-12 13:41 henry Note Added: 0007167
2016-11-12 16:52 henry Note Added: 0007168
2016-11-12 18:41 henry Note Added: 0007169
2016-11-12 21:40 alexeym Note Added: 0007170
2016-11-12 22:13 henry Note Added: 0007171
2016-11-13 09:14 alexeym File Added: wmake-quiet-cosmetic-changes.patch
2016-11-13 09:14 alexeym Note Added: 0007172
2016-11-13 09:52 henry Note Added: 0007173
2016-11-13 10:07 alexeym File Added: build.log
2016-11-13 10:07 alexeym Note Added: 0007174
2016-11-13 10:21 henry Note Added: 0007175
2016-11-13 11:02 alexeym Note Added: 0007176
2016-11-13 11:15 henry Note Added: 0007177
2016-11-13 11:22 henry Note Added: 0007178
2016-11-13 15:26 alexeym File Added: 0001-Colon-alignment.patch
2016-11-13 15:26 alexeym Note Added: 0007182
2016-11-13 15:39 henry Note Added: 0007185
2016-11-13 15:44 henry Note Added: 0007186
2016-11-13 15:49 henry Note Added: 0007187
2016-11-13 18:14 henry Note Added: 0007192
2016-11-13 18:56 alexeym File Added: 0001-Messages-and-alignment.patch
2016-11-13 18:56 alexeym Note Added: 0007193
2016-11-13 20:13 henry Note Added: 0007194
2016-11-13 21:50 alexeym Note Added: 0007197
2016-11-13 22:04 henry Assigned To => henry
2016-11-13 22:04 henry Status new => resolved
2016-11-13 22:04 henry Resolution open => fixed
2016-11-13 22:04 henry Fixed in Version => dev