On Mon, May 17, 2010 at 12:16 PM, Xavier Chantry
<[email protected]> wrote:
> On Mon, May 17, 2010 at 6:15 PM, Marc - A. Dahlhaus <[email protected]> wrote:
>>
>> It would be very nice if Xavier or you could take a look at it as i
>> don't want to step on anybodies foots along the way if i change this...
>>
>
> You really wouldn't. When you spot a regression caused by a commit
> that doesn't mention the regression anywhere (commit log or in the
> code), it's almost certainly unwanted. So a patch is welcome :)
> It's just as Dan said, the only use of Include I ever see if to
> include mirror list in Server section, so I overlooked your usage.
>
> Can you please test the attached untested patch ?
This proved that it worked, but didn't prove that Include still works
elsewhere. I'm going to see if I can wire up pactest for that.
$ cat pactest/tests/config001.py
self.description = "Quick check for Include being parsed in [options]"
self.option['Include'] = ['/dev/null']
p = pmpkg("foobar")
p.files = ["bin/foobar"]
p.desc = "test description"
p.groups = ["foo"]
p.url = "http://www.archlinux.org";
p.license = "GPL2"
p.arch = "i686"
self.addpkg2db("local", p)
self.args = "-Qi %s" % p.name
self.addrule("PACMAN_RETCODE=0")
