Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Would you consider joining the gang and Support placing the third-party library in /sites/all/libraries?
I'm using ARC also - for XMP extraction of metadata embedded in images - and would like to avoid doubling up on things like this.
I know path_to_arc can easily be hand-configured, but just thought I'd bring this new and popular approach to your attention to try.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 595974-rdf-libraries-D6.patch | 911 bytes | Dave Reid |
| #2 | 595974-rdf-libraries-dependency-D6.patch | 1.16 KB | Dave Reid |
Comments
Comment #1
dman CreditAttribution: dman commentedWe have a tag for this!
Comment #2
Dave ReidTwo different patches. One takes an optional approach to Libraries API integration and uses it only if the module is enabled. Other approach creates a dependency on the Libraries API module, which I'd encourage and prefer. I'm actually using the dependency version on my production sites and it's much easier to update the module since I don't have to keep the ARC code inside the module's folder.
Comment #3
dman CreditAttribution: dman commentedBoth fine patches.
I'm wondering if - now that the path is configurable - whether that should be a variable or the resolution should take place closer to the include - rather than a constant.
But I guess because of tight loops and all, it's better out there like that. No practical reason NOT to make it so. I just feel funny when I see constants declared from calculated variables. ... just personal style I guess.
I generally prefer the soft approach, rather than yet another requirement (libraries API is still -dev! ), but maybe making libraries API required will speed its uptake - which I'm behind. libraries API is the new 'token.module' - a shoe-in for core.
So ... generally all good. Patches work. Commit either, I say.
I won't RTBC it yet, as there may be opinion still..
Comment #4
scor CreditAttribution: scor commentedgood idea. thanks. http://drupal.org/cvs?commit=298484
Comment #6
ClearXS CreditAttribution: ClearXS commentedI don't know what is fixed, but apparently not the initial request
...as I have the latest dev while as well the project page as the INSTALL.txt say that it has to go into a module sub-dir.
Comment #7
febbraro CreditAttribution: febbraro commentedOk, this is now in the alpha8 release.
Comment #8
pcwick CreditAttribution: pcwick commenteddeleted