MAC OS X: README.html contains invalid links for locally bundled files.
|Reported by:||macca8||Owned by:||criezy|
|Priority:||normal||Component:||Port: Mac OS X|
This bug is directly related to changes made to the README.md file by commit e271cdc, when it was merged as part of the RISCOS-related PR1587 on 18 April 2019.
Prior to this commit, the ‘Credits’ section of the README file contained a link to ScummVM’s online Credits page (<https://www.scummvm.org/credits/>).
This changes that link to target the locally bundled AUTHORS file.
Mac OS X daily builds are bundled with the improved AUTHORS.rtf file, not the plain text AUTHORS file, so this link returns a ‘File not found’ error in the web browser.
In addition, a new Section 11 (‘Changelog’) has been added to the README file, which contains a link that targets the locally bundled NEWS.md file.
Mac OS X daily builds are bundled with both a plain text NEWS.md file and a NEWS.html file. In this case, the link returns the plain text version, instead of the properly formatted html version (which actively links to ScummVM’s online Commit page).
Both of these files (AUTHORS.rtf & NEWS.html) are easily, and correctly, accessed from the ScummVM app’s Help Menu, so I don’t see any value duplicating these links in the README.html file.
The online Credits page link at least offers a version with navigation links, and doesn’t require TextEdit to launch, so it’s worth retaining in the README.
Personally, I’d prefer to see the commit reverted, at least as far as Mac OS X is concerned.
Current Daily Build: 2.1.0git5224-gc84943cbc7(24 April 2019)
Platform: Intel Mac (OS X 10.6.8, 10.11.6)