Merge pull request #14 from charsbar/stringify_version

Make sure to stringify version

6 files changed, 89 insertions, 23 deletions

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
new file mode 100644
index 0000000..9ae21c1
--- /dev/null
+++ b/.github/workflows/build.yml
@@ -0,0 +1,35 @@
+name: build
+
+on:
+  push:
+    branches:
+      - '*'
+    tags-ignore:
+      - '*'
+  pull_request:
+
+jobs:
+  perl:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        perl-version:
+          - '5.8'
+          - '5.10'
+          - '5.18'
+          - '5.20'
+          - '5.26'
+          - 'latest'
+
+    container:
+      image: perl:${{ matrix.perl-version }}
+
+    steps:
+      - uses: actions/checkout@v1
+      - name: perl -V
+        run: perl -V
+      - name: Install dependencies
+        run: curl -sL https://git.io/cpm | perl - install -g --with-recommends --with-test --with-configure --show-build-log-on-failure
+      - name: Run tests
+        run: perl Makefile.PL && make && make test
diff --git a/.github/workflows/build_mac.yml b/.github/workflows/build_mac.yml
new file mode 100644
index 0000000..2d9e7f0
--- /dev/null
+++ b/.github/workflows/build_mac.yml
@@ -0,0 +1,24 @@
+name: build_mac
+
+on:
+  push:
+    branches:
+      - '*'
+    tags-ignore:
+      - '*'
+  pull_request:
+
+jobs:
+  perl:
+    runs-on: macOS-latest
+
+    steps:
+      - uses: actions/checkout@master
+      - name: perl -V
+        run: perl -V
+      - name: Install dependencies with develop
+        run: curl -sL https://git.io/cpm | perl - install -g --with-recommends --with-test --with-configure --with-develop --show-build-log-on-failure
+      - name: Run Makefile.PL
+        run: perl Makefile.PL
+      - name: Run tests
+        run: make && make test
diff --git a/.github/workflows/build_windows.yml b/.github/workflows/build_windows.yml
new file mode 100644
index 0000000..1cc197a
--- /dev/null
+++ b/.github/workflows/build_windows.yml
@@ -0,0 +1,28 @@
+name: build_windows
+
+on:
+  push:
+    branches:
+      - '*'
+    tags-ignore:
+      - '*'
+  pull_request:
+
+jobs:
+  perl:
+    runs-on: windows-latest
+
+    steps:
+      - uses: actions/checkout@master
+#      - name: Set up Perl
+#        run: |
+#          choco install strawberryperl
+#          echo "##[add-path]C:\strawberry\c\bin;C:\strawberry\perl\site\bin;C:\strawberry\perl\bin"
+      - name: perl -V
+        run: perl -V
+      - name: Install dependencies with develop
+        run: curl -sL https://git.io/cpm | perl - install -g --with-recommends --with-test --with-configure --with-develop --show-build-log-on-failure
+      - name: Run Makefile.PL
+        run: perl Makefile.PL
+      - name: Run tests
+        run: gmake test
diff --git a/.travis.yml b/.travis.yml
deleted file mode 100644
index 7610fa0..0000000
--- a/.travis.yml
+++ /dev/null
@@ -1,22 +0,0 @@
-language: perl
-perl:
-  - 5.22
-  - "5.20"
-  - 5.18
-  - 5.16
-  - 5.14
-  - 5.12
-  - "5.10"
-  - 5.8
-before_install:
-  - perlbrew install-cpanm -f
-install:
-  - cpanm -nq --installdeps --with-develop --with-recommends --with-suggests .
-script:
-  - prove -lr t xt
-  - cpanm -nq version
-  - prove -lr t xt
-env:
-  - PERL5LIB=
-  - PERL5LIB=t/fatlib_9909
-  - PERL5LIB=t/fatlib_9913
diff --git a/cpanfile b/cpanfile
index 214f5a2..b8b4226 100644
--- a/cpanfile
+++ b/cpanfile
@@ -8,7 +8,7 @@ configure_requires 'ExtUtils::MakeMaker::CPANfile' => '0.07';
 
 on test => sub {
   requires 'File::Temp' => '0.19'; # newdir
-  requires 'Test::More' => '0.88';
+  requires 'Test::More' => '0.94';
 };
 
 on develop => sub {
diff --git a/lib/Parse/PMFile.pm b/lib/Parse/PMFile.pm
index 11736d7..cbea781 100644
--- a/lib/Parse/PMFile.pm
+++ b/lib/Parse/PMFile.pm
@@ -462,6 +462,7 @@ sub _packages_per_pmfile {
                             length($ppp->{$pkg}{version});
             }
             $ppp->{$pkg}{filemtime} = $filemtime;
+            $ppp->{$pkg}{version} .= "";    # make sure to stringify version
         } else {
             # $self->_verbose(2,"no pkg found");
         }