Stephen Finucane <step...@that.guru> writes: > Now that we're pinning versions, we're going to see more frequent > dependency version changes. Requiring a rebuild after every one of these > is tiresome so don't force it and instead display a helpful message > merely suggesting that a rebuild may be necessary. > > Signed-off-by: Stephen Finucane <step...@that.guru> > Cc: Daniel Axtens <d...@axtens.net> > --- > This is a modified version of an earlier patch I submitted to the list. > The original was rejected as there were concerns that it would lead to > some confusion. However, I think the warning it prints is explicit > enough that someone can make their own mind up about this.
OK, fair enough. Acked-by: Daniel Axtens <d...@axtens.net> Regards, Daniel > --- > tools/docker/entrypoint.sh | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/tools/docker/entrypoint.sh b/tools/docker/entrypoint.sh > index 997b8763..afa85333 100755 > --- a/tools/docker/entrypoint.sh > +++ b/tools/docker/entrypoint.sh > @@ -69,9 +69,10 @@ fi > for x in /tmp/requirements-*.txt; do > if ! cmp $x ~/patchwork/$(basename $x); then > echo "A requirements file has changed." > - echo "Please rebuild the patchwork image:" > + echo "You may need to rebuild the patchwork image:" > echo " docker-compose build web" > - exit 1 > + echo "" > + diff -u $x ~/patchwork/$(basename $x) > fi > done > > -- > 2.17.1 _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork
