Opened 17 years ago
Last modified 16 years ago
#13825 new defect
port lint should check non-selected variants and platforms
Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | macports-tickets@… |
---|---|---|---|
Priority: | Normal | Milestone: | MacPorts Future |
Component: | base | Version: | 1.6.0 |
Keywords: | Cc: | ||
Port: |
Description
port lint
checks patchfile names (see #12958) but only in variants or platforms which are selected. For example, if a portfile includes
platform darwin 6 { patchfiles foo }
and one is not running Mac OS X 10.2, then no warning is issued for the patchfile foo, though one should be. Or if a portfile includes
variant abc description abc { patchfiles bar }
and one does not run sudo port lint +abc
(and +abc is not a default variant), then no warning issued for the patchfile bar, though one should be.
Change History (3)
comment:1 Changed 17 years ago by afb@…
Summary: | port lint should check patchfile names in non-selected variants and platforms → port lint should check non-selected variants and platforms |
---|
comment:2 Changed 17 years ago by ryandesign (Ryan Carsten Schmidt)
Of course. I mean, one phase of lint simply does a line-by-line check of the portfile, so that phase already checks inactive variants. But whatever the second phase does (including checking patchfiles as reported in this ticket and checking port dependencies as reported in #14380) should check inactive variants as well, somehow.
comment:3 Changed 16 years ago by tobypeterson
Milestone: | MacPorts base bugs → MacPorts Future |
---|
Milestone MacPorts base bugs deleted
Proabably should be made more generic, so that it runs all checks on inactive variants ?