For the Drupal 8 build, consider using composer to bring in the dependent library for fontawesome with this repo: https://packagist.org/packages/components/font-awesome

composer require components/fontawesome:~4 should be sufficient.

Comments

cosmicdreams created an issue. See original summary.

d34dman’s picture

If we are going to provide means to install and enable fontawesome then we can eventually remove dependency on that module too, as discussed in this issue - #2779197: Remove dependency on fontawesome

geerlingguy’s picture

Yes, please :)

I've found that the fontawesome.module includes the library in a way that is broken with the latest version of Drupal Core (due to a Libraries API module quirk in 3.x-dev), so if we can make this module run without external dependencies, that would be ideal.

I'm going to post a patch soon...

truls1502’s picture

Status: Active » Postponed (maintainer needs more info)
Issue tags: +postponed2w

Please let me know if you have worked with a patch to this issue?

geerlingguy’s picture

Sorry, I'm not working the project for which I was using this module anymore, and I must've let the posting of this patch completely slip my mind. It seems like a case where someone who is motivated could add the dep by doing a composer init to create a new composer.json with project type drupal-project (I think), and add the dependency as mentioned earlier in this thread.

Then, when you install the module via composer, it would automatically add the dependency. The module might need to have the path references for fontawesome be changed a little too.

truls1502’s picture

I do not know if it makes sense to have the composer on 7.x version module? In 8.x, yes, but this issue is related to 7.x. :)

geerlingguy’s picture

Version: 7.x-1.x-dev » 8.x-1.x-dev

At the time this issue was created, there was no 8.x-1.x-dev branch; updating the issue metadata to match :)

truls1502’s picture

Status: Postponed (maintainer needs more info) » Active
Issue tags: -postponed2w
d34dman’s picture

Version: 8.x-1.x-dev » 3.x-dev
Status: Active » Closed (won't fix)