In the latest release tag (7.x-2.0-beta4) the library definition for xmlseclibs has been removed, however it is still required as a dependency. This results in an error on the status report page. (See image.)
The removed code seems out of context for the commit, so I assume it is a mistake. I will make a patch to add the libraries definition again.
If the library is in fact no longer needed, then please inform me and I will make a patch to remove the code declaring the dependency requirement.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | saml_sp-2870680-2-add-xmlseclibs-library-definition.patch | 701 bytes | tss |
| Screen Shot 2017-04-18 at 14.41.51.png | 11.74 KB | tss |
Comments
Comment #2
tss commentedAdded patch.
Comment #3
jrochate commentedok, that fixes the message, but...
... on php-saml lib definition (saml_sp.module, line 267) the lib is already included.
Maybe the dependency should be simply removed as a lib, since it seams already included in php-saml as "2.0.0 modified" version.
Comment #4
jrglasgow commentedI intentionally removed the library definition for xmlseclibs, but I forgot to remove the section of the requirements page. I have made that change.
Comment #5
victorlap@outlook.com commentedCan you also update the readme page? On there it still says to download the xmlseclibs
And maybe also remove the following code, it throws an error on the status page: