Message56785
| Author |
christian.heimes |
| Recipients |
christian.heimes, draghuram, sapetnioc |
| Date |
2007年10月26日.11:45:56 |
| SpamBayes Score |
0.02951717 |
| Marked as misclassified |
No |
| Message-id |
<4721D36C.9040201@cheimes.de> |
| In-reply-to |
<1193398778.75.0.0834754771338.issue1322@psf.upfronthosting.co.za> |
| Content |
> I am writing a patch but I have a few questions:
>
> 1) There are at most three places where the distribution name can be
> found. What is the priority order to select only one name ?
> The three places are:
> a) Inside the /etc/lsb-release file
> b) In the name of the /etc/<distrib>-release file
> c) In the content of the /etc/<distrib>-release file
As far as I remember the specs a /etc/*-release file has a higher
priority than /etc/lsb-release.
> 2) Can I remove supported_dists parameter of platform.dist ?
> There could be a list of supported distributions but why
> as a parameter of this function ?
I agree. A module global list is better than a list as a function argument.
Can you also use a global variable instead of "/etc"? Something like
ETC_DIR = "/etc" for example. It would allow you to ship samples from
several distribution and run unit tests against each.
Christian |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2007年10月26日 11:45:57 | christian.heimes | set | spambayes_score: 0.0295172 -> 0.02951717 recipients:
+ christian.heimes, draghuram, sapetnioc |
| 2007年10月26日 11:45:56 | christian.heimes | link | issue1322 messages |
| 2007年10月26日 11:45:56 | christian.heimes | create |
|