Add oyejorge/less.php as default compiler

Using the latest bootstrap, I was having lessphp compile errors along the lines of:
Parse error: failed at &:extend(.clearfix all);

As per this issue report on github the gpeasy.com (aka as oyejorge/less.php) does indeed solve this particular problem, so +1 to @kyletaylored's suggestion to switch!

http://lessphp.gpeasy.com/#integration-with-other-projects

This library can be used as drop-in replacement of lessphp to work with Drupal 7 less module.

How to install:

Download the less.php source code and unzip it so that 'lessc.inc.php' is located at 'sites/all/libraries/lessphp/lessc.inc.php'.
Download and install Drupal 7 less module as usual.
That's it :)

This worked for me.

Attached is a patch to update the less modules docs and hook_requirements() with the more reliable LessPHP compiler from gpeasy.

Someone will also need to update the Less module project page too.

I'm in the process of adding in multi-compiler support.

I will be directing users to visit the project page for a list of compatible compilers and making the hard-coded documentation within the module itself more generic since its easier to make a change to the project page than it is to the module code.

This is related to #2003628: Add ability to use node.js version of LESS.

Its related from a code perspective.

I'm changing some of the wrapper code that handles which library to use, and adding support for oyejorge/less.php.

When I had last checked there were issues with complete functional parity between leafo and oyejorge.

I will test again and update the project page to reflect my results.

The update to the project page will not wait until after the update to the module code if everything is working, but it will wait till after I'm not at work.

I agree, there needs to be support for both (backwards compatible and existing projects that use Leafo/lessphp).

Transitioning from Leafo/lessphp
...
Note, the 'setPreserveComments', 'registerFunction' and 'unregisterFunction' will no longer have any effect on the compiled less.

That being said, it should probably be the default moving forward. FYI, it would probably be beneficial to also add Composer support http://lessphp.gpeasy.com/#installation

Spent several hours going through and adding support for oyejorge/less.php, and #2203549.

Ran into an issue that prevent source maps from being generated correctly.

Logged the issue at the following:

https://github.com/oyejorge/less.php/issues/127

Once that issue is resolved, I will be able to complete this functionality.

I've also changed the links on the project page to point to http://lessphp.gpeasy.com/#integration-with-other-projects

Documentation pages have also been pointed to oyejorge/less.php.

I've updated the information in composer.json to use references to oyejorge/less.php.

I've also attempted to use the following to have the default install path be the libraries folder:

  "extra": {
    "installer-paths": {
      "sites/all/libraries/less.php": ["vendor/package"]
    }
  }

I have yet to get it to actually install to that location.

Is this a Drush only extension or something?

Any help on correctly configuring and testing changes to composer.json would be appreciated.

Marking this as fixed as 7.x-4.0 will be released soon.

Documentation has been updated in the README and on documentation pages.

Composer support has also been updated.

Additionally, support for leafo/lessphp has not been removed.

If there are any other issues with integration between the module and one of the PHP engines, please open a new ticket.

leafo/lessphp is still default despite it not being recommended. Perhaps change default to the library on http://lessphp.gpeasy.com/.

This is in latest stable version.