Yes I am angry! People put stuff into next that is broken and obviously not 
tested and leave it there for days even though they get email every day telling 
them about the failed builds and they should also be checking the dashboard the 
day after they push something. If you can't test your merged next before 
pushing and are not willing to fix problems that come up in the nightly the 
very next day then we are going to have to turn off next access to everyone but 
Satish and make everyone use pull requests for all code!

    I don't think people want this and it will slow down development a great 
deal but what choice is there?

    The reason you have to fix your own mess is that only you know what caused 
the mess and what the correct fix is. Reverting out people's bad commits to 
next is cumbersome and error prone due to Git's design so it is not trivial to 
just flush next whenever some irresponsible person messes up next.

     So fix these problems NOW! And don't do it again. Run make alltests with a 
couple of configure options such as complex and 64 bit indices to catch the 
major problems before pushing to next!

     Note: I don't expect that when you push to next every single test machine 
in the nightly build will be perfect; next is exactly for catching the corner 
cases that only appear on  strange systems. But I do expect that your pushes 
will be clean on a majority of the machines and that you will fix problems that 
come up promptly.

  Barry



[petsc-checkbuilds] PETSc blame digest (next) 2016-10-15

 src/snes/examples/tutorials/ex56.c:400
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-opt_crank.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-opt_crank.log>]
     /sandbox/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' undeclared (first use in this function)
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-valgrind_el6.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-valgrind_el6.log>]
     /sandbox/petsc/petsc.clone-4/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' undeclared (first use in this function)
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-pkgs-opt_wii.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-pkgs-opt_wii.log>]
     /usr/home/balay/petsc.clone/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' undeclared (first use in this function)
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-xsdk-opt_ipro.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-xsdk-opt_ipro.log>]
     /Users/petsc/petsc.clone-4/src/snes/examples/tutorials/ex56.c:400:7: 
error: use of undeclared identifier 'two_solves'
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-threadcomm_el6.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-threadcomm_el6.log>]
     /sandbox/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' undeclared (first use in this function)
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6_ipro.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6_ipro.log>]
     /Users/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c:400:7: error: 
use of undeclared identifier 'two_solves'
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-64idx_thrash.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-64idx_thrash.log>]
     /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' undeclared (first use in this function)
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6-cxx-pkgs-opt_ipro.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6-cxx-pkgs-opt_ipro.log>]
     /Users/petsc/petsc.clone-2/src/snes/examples/tutorials/ex56.c:400:7: 
error: use of undeclared identifier 'two_solves'
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-cxx-mlib_el6.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-cxx-mlib_el6.log>]
     /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' was not declared in this scope
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-cxx-pkgs-opt_wii.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-cxx-pkgs-opt_wii.log>]
     /usr/home/balay/petsc.clone-2/src/snes/examples/tutorials/ex56.c:400:7: 
error: 'two_solves' was not declared in this scope
   
[http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-dbg-ftn-interfaces_crank.log
 
<http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-dbg-ftn-interfaces_crank.log>]

> 
> 
> warnings attributed to commit 
> https://bitbucket.org/petsc/petsc/commits/4e26cc2
> DM: switch to using PetscDS boundary in examples
> 
>  src/snes/examples/tutorials/ex56.c:346
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-xsdk-opt_ipro.log]
>      /Users/petsc/petsc.clone-4/src/snes/examples/tutorials/ex56.c:346:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-opensolaris-pkgs-opt_n-gage.log]
>      "/export/home/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c", 
> line 346: warning: enum type mismatch: arg #2
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6_ipro.log]
>      /Users/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c:346:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-cxx-mlib_el6.log]
>      /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:346:123: 
> error: cannot convert 'PetscBool' to 'DMBoundaryConditionType' for argument 
> '2' to 'PetscErrorCode PetscDSAddBoundary(PetscDS, DMBoundaryConditionType, 
> const char*, const char*, PetscInt, PetscInt, const PetscInt*, void (*)(), 
> PetscInt, const PetscInt*, void*)'
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-dbg-ftn-interfaces_crank.log]
>      /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:346:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-cxx-pkgs-opt_wii.log]
>      
> /usr/home/balay/petsc.clone-2/src/snes/examples/tutorials/ex56.c:346:123: 
> error: cannot convert 'PetscBool' to 'DMBoundaryConditionType' for argument 
> '2' to 'PetscErrorCode PetscDSAddBoundary(PetscDS, DMBoundaryConditionType, 
> const char*, const char*, PetscInt, PetscInt, const PetscInt*, void (*)(), 
> PetscInt, const PetscInt*, void*)'
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6-cxx-pkgs-opt_ipro.log]
>      /Users/petsc/petsc.clone-2/src/snes/examples/tutorials/ex56.c:346:12: 
> error: no matching function for call to 'PetscDSAddBoundary'
> 
>  src/snes/examples/tutorials/ex56.c:347
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-xsdk-opt_ipro.log]
>      /Users/petsc/petsc.clone-4/src/snes/examples/tutorials/ex56.c:347:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-opensolaris-pkgs-opt_n-gage.log]
>      "/export/home/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c", 
> line 347: warning: enum type mismatch: arg #2
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6_ipro.log]
>      /Users/petsc/petsc.clone/src/snes/examples/tutorials/ex56.c:347:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-osx-10.6-cxx-pkgs-opt_ipro.log]
>      /Users/petsc/petsc.clone-2/src/snes/examples/tutorials/ex56.c:347:12: 
> error: no matching function for call to 'PetscDSAddBoundary'
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-dbg-ftn-interfaces_crank.log]
>      /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:347:37: 
> warning: implicit conversion from enumeration type 'PetscBool' to different 
> enumeration type 'DMBoundaryConditionType' [-Wenum-conversion]
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-freebsd-cxx-pkgs-opt_wii.log]
>      
> /usr/home/balay/petsc.clone-2/src/snes/examples/tutorials/ex56.c:347:114: 
> error: cannot convert 'PetscBool' to 'DMBoundaryConditionType' for argument 
> '2' to 'PetscErrorCode PetscDSAddBoundary(PetscDS, DMBoundaryConditionType, 
> const char*, const char*, PetscInt, PetscInt, const PetscInt*, void (*)(), 
> PetscInt, const PetscInt*, void*)'
>    
> [http://ftp.mcs.anl.gov/pub/petsc/nightlylogs/archive/2016/10/15/examples_next_arch-linux-pkgs-cxx-mlib_el6.log]
>      /sandbox/petsc/petsc.clone-3/src/snes/examples/tutorials/ex56.c:347:114: 
> error: cannot convert 'PetscBool' to 'DMBoundaryConditionType' for argument 
> '2' to 'PetscErrorCode PetscDSAddBoundary(PetscDS, DMBoundaryConditionType, 
> const char*, const char*, PetscInt, PetscInt, const PetscInt*, void (*)(), 
> PetscInt, const PetscInt*, void*)'
> 
> ----
> To opt-out from receiving these messages - send a request to 
> petsc-dev@mcs.anl.gov.

Reply via email to