---
- branch: MAIN
  date: Wed Aug 10 21:48:48 UTC 2022
  files:
  - new: '1.71'
    old: '1.70'
    path: pkgsrc/pkgtools/lintpkgsrc/files/lintpkgsrc.pl
    pathrev: pkgsrc/pkgtools/lintpkgsrc/files/lintpkgsrc.pl@1.71
    type: modified
  - new: '1.7'
    old: '1.6'
    path: pkgsrc/pkgtools/lintpkgsrc/files/t/glob.t
    pathrev: pkgsrc/pkgtools/lintpkgsrc/files/t/glob.t@1.7
    type: modified
  - new: '1.6'
    old: '1.5'
    path: pkgsrc/pkgtools/lintpkgsrc/files/t/parse_makefile.t
    pathrev: pkgsrc/pkgtools/lintpkgsrc/files/t/parse_makefile.t@1.6
    type: modified
  id: 20220810T214848Z.b2b57ba28b525435f477cb12a76dd717ada59079
  log: |
    lintpkgsrc: fix evaluation of conditions in makefiles

    Previously, evaluating the conditions was a wild mixture of concepts
    taken from bmake's cond.c, without attention to detail. For example, in
    the condition 'exists(VAR:Mpattern)', the ':Mpattern' doesn't make any
    sense, still lintpkgsrc parsed it as a modifier.

    Instead of evaluating the ':M' modifier only in conditions, a better
    idea is to have a general subroutine that expands an arbitrary strings,
    taking care of all kinds of modifiers.

    While the previous code was conceptually wrong, the number of practical
    difference is small. Still, it's better to be precise and accurate,
    there are enough places with bad heuristics in the rest of the
    lintpkgsrc code.
  module: pkgsrc
  subject: 'CVS commit: pkgsrc/pkgtools/lintpkgsrc/files'
  unixtime: '1660168128'
  user: rillig