Bart Smaalders wrote: ... snippety-doo-dah...
Lines 137,141: Would it be possible to wrap these in a test of the value of the 'ftpuser' attribute or do you envisage a situation where some earlier call could add a user to ftpusers, and a later call would say to not have this user in ftpusers?This works now w/ adding and removing ftpuser capability. What were you trying to achieve?
Never mind - I was just thinking of trying to eliminate a code path if it was not necessary to execute it - e.g.
if final_attrs['ftpuser']:
ftp.setuser(...)
ftp.writefile()
But it probably adds very little value.
Cheers,
Trev
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
