Merge pull request #5 from PDLPorters/gha-shared-actions

Use shared GitHub Actions

.github/workflows/ci.yml

@@ -0,0 +1,70 @@
+name: perl
+on:
+  push:
+    branches:
+      - '*'
+    tags-ignore:
+      - '*'
+  pull_request:
+  create:
+jobs:
+  notify:
+    runs-on: ubuntu-latest
+    continue-on-error: true
+    if: ${{ always() }}
+    steps:
+      - uses: PDLPorters/devops/github-actions/irc-notifications@master
+        with:
+          target-notifications: true
+  ci:
+    runs-on: ${{ matrix.os }}
+    if: ${{ github.event_name == 'push' || github.event_name == 'pull_request' }}
+    needs: notify
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ubuntu-latest]
+        perl-version: ['5.10', '5.14', '5.20']
+        include:
+          - perl-version: '5.30'
+            os: ubuntu-latest
+            release-test: true
+            coverage: true
+          #- perl-version: '5.30'
+          #  os: windows-latest
+          - perl-version: '5.30'
+            os: macos-11
+    steps:
+      - uses: actions/checkout@v2
+      - name: 'ci-dist: target-setup-perl'
+        uses: PDLPorters/devops/github-actions/ci-dist@master
+        with:
+          target-setup-perl: true
+          perl-version: ${{ matrix.perl-version }}
+      - name: Install PDL dependencies
+        uses: PDLPorters/devops/github-actions/install-dep-pdl-dep@master
+      - name: Install Perl configure deps
+        shell: bash
+        run: |
+          $MYPERL -S cpanm --dev -n Alien::Base::ModuleBuild || cat ~/.cpanm/build.log
+      - name: 'ci-dist: target-all'
+        uses: PDLPorters/devops/github-actions/ci-dist@master
+        with:
+          target-setup-perl: false
+          target-install-dist-perl-deps: true
+          target-test-release-testing: true
+          target-test: true
+          test-enable-release-testing: ${{ matrix.release-test }}
+          test-enable-coverage: ${{ matrix.coverage }}
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+
+  build-status:
+    runs-on: ubuntu-latest
+    continue-on-error: true
+    if: ${{ always() }}
+    needs: [ 'ci' ]
+    steps:
+      - uses: PDLPorters/devops/github-actions/irc-notifications@master
+        with:
+          target-build-status: true
+          needs: ${{ toJSON(needs) }}

.github/workflows/issue-notify.yml

@@ -0,0 +1,17 @@
+name: issue-notify
+
+on:
+  issues:
+    types: [opened]
+  issue_comment:
+    types: [created]
+
+jobs:
+  notify:
+    runs-on: ubuntu-latest
+    continue-on-error: true
+    if: ${{ always() }}
+    steps:
+      - uses: PDLPorters/devops/github-actions/irc-notifications@master
+        with:
+          target-notifications: true

Build.PL

@@ -37,6 +37,7 @@ if (my $s = `pkg-config --variable pc_path pkg-config`) {
   }
 
   $dest_libdir = $first_lib || $first_lib64 || "";
+  undef $dest_libdir if $> != 0; # not root
   $prefix = "%s";
 
   if($dest_libdir) {
@@ -45,10 +46,8 @@ if (my $s = `pkg-config --variable pc_path pkg-config`) {
   	$prefix = $s;
       }
   } else {
-      if($first_share_pc) {
+      if($first_share_pc && $> == 0) {
   	$bonus_install_command = "mv %s/lib/pkgconfig/fftw*.pc $first_share_pc";
-      } else {
-  	die "Couldn't figure out a reasonable place to stick the pkg-config files for fftw.\n  Sorry, you're on your own.";
       }
   }
 }  

lib/Alien/FFTW3.pm

@@ -76,8 +76,15 @@ use warnings;
 our $VERSION = '0.04';
 use parent 'Alien::Base';
 
+use Env qw(@PKG_CONFIG_PATH);
+use File::Spec;
+
 our $pkgconfig;
 BEGIN {
+   unshift @PKG_CONFIG_PATH, File::Spec->catfile(
+     __PACKAGE__->dist_dir,
+     qw(lib pkgconfig)
+   ) if __PACKAGE__->install_type('share');
    if ($^O eq 'MSWin32') {
      # no 'which' on MS Windows but 'pkg-config' might be installed
      $pkgconfig = 'pkg-config' if `pkg-config --version`;