[PATCH 2/3] Add check testing visibility of password entries

Vratislav Podzimek vpodzime at redhat.com
Mon Dec 2 16:32:36 UTC 2013


On Mon, 2013-12-02 at 10:44 -0500, David Shea wrote:
> On 11/29/2013 06:06 AM, Vratislav Podzimek wrote:
> > diff --git a/tests/Makefile.am b/tests/Makefile.am
> > index 2feab20..0dd998a 100644
> > --- a/tests/Makefile.am
> > +++ b/tests/Makefile.am
> > @@ -38,6 +38,7 @@ dist_check_DATA = pylint/pylint-false-positives
> >   # Test scripts need to be listed both here and in TESTS
> >   dist_check_SCRIPTS = glade/accelerators/check_accelerators.py \
> >   		     glade/accelerators/run_check_accelerators.sh \
> > +             glade/pw_visibility/run_check_pw_visibility.sh \
> >   		     nosetests.sh \
> >   		     pylint/intl.py \
> >   		     pylint/preconf.py \
> You also need to add glade/pw_visibility/check_pw_visibility.py to this 
> variable. And when running make check, pylint complains about succ and 
> fpath being redefined in check_pw_visibility.py. Otherwise, looks great!
Good points, thanks! Fixing locally.

-- 
Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic



More information about the anaconda-patches mailing list