Installation
============
If you are using the rest server you will need to download the latest version of SPYC:
wget https://raw.github.com/mustangostang/spyc/79f61969f63ee77e0d9460bc254a27... -O servers/rest_server/lib/spyc.php

This is just wrong ... the file must go in libraries/spyc

CommentFileSizeAuthor
#3 update_the_readme_for-2760873-3.patch639 bytesmilsyobtaf
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jmary created an issue. See original summary.

tyler.frankenstein’s picture

tyler.frankenstein CreditAttribution: tyler.frankenstein commented
Title: Update the README.txt » Update the README for spyc location
Version: 7.x-3.15 » 7.x-3.x-dev
Category: Bug report » Task
Priority: Critical » Normal
milsyobtaf’s picture

milsyobtaf CreditAttribution: milsyobtaf at Acquia commented
FileSize
update_the_readme_for-2760873-3.patch639 bytes

In addition to the target, the location of the download needs to be updated to reflect the current make file. Patch included.

tyler.frankenstein’s picture

tyler.frankenstein CreditAttribution: tyler.frankenstein commented
Status: Active » Fixed

Thank you!

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

Actually, need to edit or remove the text under this changed line:

Once downloaded you need to add spyc.php to the rest_server/lib folder which exists under the location you have installed services in.

tyler.frankenstein’s picture

tyler.frankenstein CreditAttribution: tyler.frankenstein commented
Status: Fixed » Needs work

@Chris, what text would you like to see there? I haven't ever had to use the spyc.php , so I do not know what the best explanation would be. Please advise, thank you.

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

Right now it's just the phrasing. Above that text we tell users to place the lib in `libraries/spyc/` but that one blurb I quoted above says to put the lib in the module (not a best practice). That's all.

dhruveshdtripathi’s picture

dhruveshdtripathi CreditAttribution: dhruveshdtripathi at DevsAdda for OpenSense Labs commented

The README file text does not match with the standards. https://www.drupal.org/node/2181737

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

I agree @dhruveshdtripathi but that should be it's own ticket. Please file. Thank you.