Skip Menu |

This queue is for tickets about the Config-General CPAN distribution.

Report information
The Basics
Id: 79869
Status: resolved
Priority: 0/
Queue: Config-General
People
Owner: tlinden [...] cpan.org
Requestors: P [...] draigBrady.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 2.51
Fixed in: (no value)

History Show all quoted text
  Wed Sep 26 18:43:24 2012 P [...] draigBrady.com - Ticket created
Subject: IncludeGlob is not recursive
So my particular use is with tgtd where it has /etc/tgt/targets.conf include /etc/tgt/conf.d/*.conf One can then drop files into /etc/tgt/conf.d However if they in turn want to include a glob it doesn't work. I could get it working by commenting out this line in General.pm local $this->{IncludeGlob}; Not knowing perl I don't understand the full implications of doing this. It seems like the code is trying to restrict globbing to a single level. Why is that? Also any suggestions for a way to support this setup without needing to update Config-General would be much appreciated.
  Wed Oct 10 11:28:34 2012 tlinden [...] cpan.org - Taken
  Wed Oct 10 11:32:07 2012 tlinden [...] cpan.org - Correspondence added
Show quoted text
> I could get it working by commenting out this line in General.pm > local $this->{IncludeGlob};
Hm, actually I can't remember anymore, why I added this local statement there :( And you're right, works like a charm without it. Since no unittest fails without this local statement I think I'll change it in the next version. Show quoted text
> Also any suggestions for a way to support this setup without > needing to update Config-General would be much appreciated.
Well if I fix the issue, an update would be required anyway.
  Wed Oct 10 11:32:08 2012 The RT System itself - Status changed from 'new' to 'open'
  Wed Apr 30 07:39:20 2014 tlinden [...] cpan.org - Correspondence added
fixed in 2.52
  Wed Apr 30 07:39:22 2014 tlinden [...] cpan.org - Status changed from 'open' to 'resolved'