Re: [master] expose FSSet properties and methods through Storage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

Given the large amount of changes this patch set makes, I've taken the
liberty of applying the patches to my own tree and running pylint,
which resulted in the following:

************* Module storage
E0602:1454:FSSet.parseFSTab: Undefined variable 'self'
E0602:1536:FSSet.fsFreeSpace: Undefined variable 'self'
E0602:1762:FSSet.createSwapFile: Undefined variable 'self'
E0602:1785:FSSet.mkDevRoot: Undefined variable 'self'
E0602:1826:FSSet.write: Undefined variable 'self'

The problem is constructions like this one:

    def parseFSTab(self, chroot=self.rootpath):

It is not allowed to reference / use self here to specify
a default value. I've created a simple reproducer and ran it
through python to be sure this was not a false postive
and pylint is correct, this construction is not allowed.

Regards,

Hans

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux