New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added CSS::LESSp to required modules to allow successful fallback #4

Merged
merged 8 commits into from Nov 26, 2015

Conversation

Projects
None yet
2 participants
@alabamapaul
Contributor

alabamapaul commented Nov 25, 2015

  • Added the CSS::LESSp to the cpanfile as a runtime, test, and author requirement to ensure module was available for fallback when 'lessc' executable was not available. This should allow the module to install successfully on systems without the 'lessc' executable
  • Created MANIFEST.SKIP and updated MANIFEST to address issues encountered when I ran 'make dist'
  • With the v0.02 module currently on CPAN, I experienced the issue reported in RT Ticket 9194. After building a new distribution using 'make dist' I no longer have this problem. I suspect this is because I am using a newer version of inc::Module::Install that does not bundle Test::Builder

@alabamapaul alabamapaul changed the title from Added CSS::LESSp to required modules to address RT 91934 to Added CSS::LESSp to required modules to allow successful fallback Nov 25, 2015

naoya added a commit that referenced this pull request Nov 26, 2015

Merge pull request #4 from alabamapaul/master
Added CSS::LESSp to required modules to allow successful fallback

@naoya naoya merged commit d38d517 into naoya:master Nov 26, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@naoya

This comment has been minimized.

Show comment
Hide comment
@naoya

naoya Nov 26, 2015

Owner

Thanks, uploaded 0.03 to CPAN

Owner

naoya commented Nov 26, 2015

Thanks, uploaded 0.03 to CPAN

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment