Skip to content
This repository has been archived by the owner on Mar 7, 2019. It is now read-only.

Testing for Acme Alien dists #2

Merged
merged 10 commits into from
Sep 5, 2014
Merged

Testing for Acme Alien dists #2

merged 10 commits into from
Sep 5, 2014

Conversation

plicease
Copy link
Contributor

@plicease plicease commented Sep 3, 2014

These commits:

  1. Adds a pkg-config style dontpanic.pc file so that libdontpanic can be installed as a system lib and detected as such by Alien::Base and Acme-Alien-DontPanic
  2. Adds a .travis.yml for testing each push
  3. Adds a .gitignore
  4. numerous small fixes to get these things working

Note that ALIEN_FORCE=1 testing is broken because Alien::Base in CPAN will not install with that environment variable. As soon as we get that fix into CPAN, these tests should start passing.

@zmughal
Copy link
Member

zmughal commented Sep 4, 2014

Merge: Aye

@plicease
Copy link
Contributor Author

plicease commented Sep 4, 2014

Me toos: Aye

@jberger
Copy link
Member

jberger commented Sep 5, 2014

Most of this looks fine, why the new gen-dontpanic.sh and if its useful why not remove the gen-dontpanic.pl? Otherwise, aye

@plicease
Copy link
Contributor Author

plicease commented Sep 5, 2014

I had trouble getting the Perl version to work. Also I think it is better to let the autotools create the tarball (I think?) since it will use a standard process. I didn't remove the old version because I wasn't feeling especially aggressive in terms of throwing stuff out, but we can remove it.

plicease added a commit that referenced this pull request Sep 5, 2014
Testing for Acme Alien dists
@plicease plicease merged commit 910eacc into Perl5-Alien:master Sep 5, 2014
@plicease
Copy link
Contributor Author

plicease commented Sep 5, 2014

I also enabled travis for this repo

@plicease plicease deleted the tidy branch September 8, 2014 21:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants