Re: [PATCH 3/7] Add a function to get paths to loaded modules

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

 



But making isys dependend on iutil, does it seem wise? It should be the other way around, shouldn't it?

Martin

----- "Chris Lumens" <clumens@xxxxxxxxxx> wrote:

> > >+        modName = modline.split(" ", 1)[0]
> > >+        modInfo = os.popen("modinfo -F filename '%s'" %
> (modName,)).readlines()
> > >+        modPaths = [ line.strip() for line in modInfo if line!=""
> ]
> > 
> > I accidentally found out that os.popen is obsolete. So I would be
> > against using it in a new code.
> 
> They likely want you to use subprocess, which execWithCapture in
> iutil.py uses.
> 
> - Chris
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
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