Check for the PHP AMP library in hook_requirements()

Hi @mtift,

I have made a patch (plus 1) to check the existence of AMP Library in hook_requirements(). I have especially set the libraries dependency to >= 2.x. This may need more work especially on the pattern of the 'version arguments' in hook_libraries_info(). It will definitely need some improvement (PCRE RegEx!!!!). The file for checking the library version should also be checked if it is the actual file. I will attach two patches: (1) a patch that addresses the task only, (2) and one that addresses the issue and the coding standard.

I hope anyone can move on forward on this task using these preliminary patches that certain needs work.

Thanks.

Please note that I have also set the check for the library during runtime only and not during install.

Thanks.

Thanks for your effort on this @johnrosswvsu But we're not using the hook_libraries_info , libraries etc approach. We're actually using composer manager which does not need all this.

Accordingly we will not be considering your patch for future inclusion -- sorry :-( !

Our approach is that of @mtift in Drupal 8 ( see http://cgit.drupalcode.org/amp/commit/?id=fcec721 )

Accordingly I've simply:
- Uncommented @mtifts preexisting hook_requirements code for Drupal 7 ( http://cgit.drupalcode.org/amp/commit/?id=262c259 )
- changed drupal_get_library to a !class_exists (like in the Drupal 8 implementation)
- Added a link to the composer manager homepage on drupal.org

Here is a screenshot if the AMP PHP Library requirement is not met.