Opened 11 years ago

Closed 11 years ago

#10 closed enhancement (fixed)

Pod documentation of OSDetector confusing and partially wrong

Reported by: human Owned by: rplessl
Priority: minor Milestone:
Component: OS-detector Version: 1.5.2
Keywords: Cc:

Description

Hi,

IMHO the Pod documentation in OSDetector.pm is confusing and partially wrong.

Confusing: It would help if the OSDetector.conf example for OS and Compatibility would be more consistent. E.g. the Compatibility list contains "i686-debian-linux3.1" but the above examples for detecting Debian Linux all result in <CPU>-linux-debian<something>. Also there is no "linux3.1" in the detection list. Consistency enhances comprehensibility.

Wrong: dots (".") should be escaped in the strings matches used for the distribution detection. I.e. "if ($debian_version =~ /4\.0/) {..." and not "if ($debian_version =~ /4.0/)...".

Cheers

frank

Change History (1)

comment:1 Changed 11 years ago by rplessl

  • Resolution set to fixed
  • Status changed from new to closed

Added better documentation in OSDetector.pm and improve OSDetecotor.conf (regexps).

Changed in r105

Note: See TracTickets for help on using tickets.