NSRegularExpression and OSX 10.6 #171

Closed
u10int opened this Issue Jan 10, 2012 · 2 comments

Comments

Projects
None yet
2 participants
@u10int

u10int commented Jan 10, 2012

Just tried updating to the latest 2.x release and received several compiler errors in GBTokenizer.m about NSRegularExpression. I'm running OSX 10.6 and the latest stable version of Xcode (4.2), but NSRegularExpression is only available in the Foundation.framework on OSX 10.7. However, the requirements for appledoc don't state OSX 10.7 is required for compiling.

appledoc/Parsing/GBTokenizer.m:320:2: error: use of undeclared identifier 'NSRegularExpression' [3]
NSRegularExpression directiveExpression = [NSRegularExpression regularExpressionWithPattern:@"^\s@(param|result|return)" options:NSRegularExpressionDotMatchesLineSeparators error:nil];

Is there an alternative solution for compiling on OSX 10.6? I notice versions prior to 2.0 were using RegexLiteKit instead...

@tomaz

This comment has been minimized.

Show comment Hide comment
@tomaz

tomaz Jan 10, 2012

Owner

Appledoc was using RegexKitLite until recently with a user contribution that added better HeaderDoc support. In this particular patch, NSRegularExpression was used. I decided to leave if as my plan is to update regular expressions, I find 10.7 classes more reliable generally. Unfortunately this makes it 10.7 only... You should be able to comment out the code in question. I made it opt in/out with cmd line switch anyway, so as long as you compile and don't use the switch, it'll work just fine,

Owner

tomaz commented Jan 10, 2012

Appledoc was using RegexKitLite until recently with a user contribution that added better HeaderDoc support. In this particular patch, NSRegularExpression was used. I decided to leave if as my plan is to update regular expressions, I find 10.7 classes more reliable generally. Unfortunately this makes it 10.7 only... You should be able to comment out the code in question. I made it opt in/out with cmd line switch anyway, so as long as you compile and don't use the switch, it'll work just fine,

@tomaz tomaz closed this Jan 10, 2012

@u10int

This comment has been minimized.

Show comment Hide comment
@u10int

u10int Jan 10, 2012

Thanks that worked.

u10int commented Jan 10, 2012

Thanks that worked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment