locations of rdf should read rdfx. There might be some more changes needed in this file that I'm just missing.

Current:

"== Install the RDF module ==

1. Copy all the module files into a subdirectory called
sites/all/modules/rdf/ under your Drupal installation directory.

2. Go to Administer >> Site building >> Modules and enable the RDFx module and
any other module you like. You will find them in the "RDF" section.

3. Install the ARC2 library following one of these 2 options:
- run "drush rdf-download" (recommended, it will download the right
package and extract it at the right place for you.)
- manual install: download the library from
http://github.com/semsol/arc2/tarball/master and extract it in the rdf
module directory such that you end up with the following file structure:
sites/all/modules/rdf/vendor/arc/ARC2.php"

CommentFileSizeAuthor
#15 readme-fix-1162276-15.patch2.95 KBclayball
#13 readme-1162276-12.patch2.9 KBclayball
#11 readme-1162276-2.patch1.88 KBclayball
#9 readme-1162276-1.patch1.78 KBclayball

Comments

Anonymous’s picture

Anonymous (not verified) commented

Good catch, thanks for pointing it out!

#3 in the README will change once #1115692: Make libraries a dependency and deprecate vendor/arc is resolved, so I suggest we get these done in tandem.

scor’s picture

scor commented
Issue tags: +Novice

@twoflower want to roll a patch to fix README.txt?

twoflower’s picture

twoflower commented
Issue tags: -Novice

Sure, but it might take longer for you to explain to me how to do this that doing it yourself = Your choice.

scor’s picture

scor commented

@twoflower: I see it as a long term investment. we need more contributors, and it's easier to start to learn making patches on small changes.

see how to roll a patch at https://drupal.org/patch/apply

scor’s picture

scor commented
Issue tags: +Novice

I'm tagging this as "novice" as a result.

Anonymous’s picture

Anonymous (not verified) commented

twoflower, if you want me to walk you through it over skype, let me know.

clayball’s picture

clayball commented
Assigned: Unassigned » clayball

I noticed this has been idle since May 19th...
I'm a new at this and am happy to take this one on :-) Currently, my biggest hurdle is figuring out how to work with git. So, I'll assign this one to me and I'll look at the issue in #3 in the description so that the work is done in tandem (hopefully). This may take me a little while to complete :-) but should serve as good place for me to gain my footing so I can become a more active contributor.

scor’s picture

scor commented

@cwells73: welcome aboard! Take the time you need to learn git, once you know your way around it, you'll be surprised how quick it is to roll a patch! See how to create a patch and feel free to ping me or Lin on IRC or skype if you run into problems...

clayball’s picture

clayball commented
Status: Needs work » Needs review
StatusFileSize
new readme-1162276-1.patch1.78 KB

There are two issues here:

item 1. Modify RDF spelling to RDFx (RDF Extensions).
item 2. #3 in the README file (see description above).

This patch fixes item 1.

Made RDF to RDFx changes along with a spelling error. Here's the patch.

Status changed to 'needs review'.

ps. Hope I'm doing this right :-)

scor’s picture

scor commented
Status: Needs review » Needs work

Yes, that's the right to do it! if that's your first patch, congrats ;)

+++ b/README.txt
@@ -1,7 +1,7 @@
+decentralized open-world assumption. This RDF Extensions, RDFx, package for ¶

RDFx should be in parenthesis (). also note the whitespace in the end (visible if you use Dreditor for reviewing patches on drupal.org)

#3 is not ready to be changed yet, since the libraries issue is still open.

clayball’s picture

clayball commented
StatusFileSize
new readme-1162276-2.patch1.88 KB

comment #10 comments fixed.

Waiting for ARC2 library issue to be resolved:

comment #1: http://drupal.org/comment/reply/1162276#comment-4489280
issue 1124580: http://drupal.org/node/1124580

scor’s picture

scor commented

I think we could also add a note about the new requirement on the entity module and maybe encourage to install restws for RDF serialization outputs?

clayball’s picture

clayball commented
StatusFileSize
new readme-1162276-12.patch2.9 KB

Added 'Related projects' and 'Dependencies' sections. used wording from: http://drupal.org/project/rdfx
Does that make sense to include?

Also added the more descriptive RDFx: notes from http://drupal.org/project/rdfx

Am I going overboard with this?

ps. I think I finally have this patch thing down. Thanks to everyone for being so helpful!

scor’s picture

scor commented

That's a good idea. Maybe add a sentence to introduce the description of all the modules, e.g. "This project includes the following modules:"

clayball’s picture

clayball commented
StatusFileSize
new readme-fix-1162276-15.patch2.95 KB

Added wording from comment #14.

Also changed '-' bullets points to '*' when listing the modules included. I don't see any module documentation guidelines for this but figured it might be a good idea to use '*' for bullet points.

scor’s picture

scor commented
Status: Needs work » Fixed

Thanks Clay! I've committed your changes: http://drupalcode.org/project/rdfx.git/commit/78dc7f7

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

h4ck3rm1k3’s picture

h4ck3rm1k3 commented
Status: Closed (fixed) » Needs work

Hi,
the readme is still wrong.

drush rdf-download is what worked, it puts the data here:

The latest ARC2 library has been downloaded to sites/all/libraries

but the readme says
sites/all/modules/rdfx/vendor/arc which is wrong and does not work.

thanks,
mike

scor’s picture

scor commented

sites/all/modules/rdfx/vendor/arc should work, that's what I use all the time. The README should encourage one location though, the sites/all/libraries one.

h4ck3rm1k3’s picture

h4ck3rm1k3 commented
Status: Needs work » Closed (works as designed)

sites/all/modules/rdfx/vendor/arc i used a symlink to arc2, maybe that was the problem.
I used a copy instead of a symlink and it seems to work. thanks,
mike