Skip to content
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

'v' prefixed module versions don't deploy #48

Closed
wki opened this issue May 10, 2012 · 3 comments
Closed

'v' prefixed module versions don't deploy #48

wki opened this issue May 10, 2012 · 3 comments

Comments

@wki
Copy link

wki commented May 10, 2012

If a module's version is prefixed with 'v', eg. DBD::Pg, MooseX::Attribute::Chained, the initial install works. However, a later install for a deployment fails. cpanm does not recognize the given Module spec like "DBD::Pgv2.19.2". If the spec were "DBD::Pg2.19.2" it would work.

@miyagawa
Copy link
Contributor

nevermind, i got it reproduced.

@wki
Copy link
Author

wki commented May 11, 2012

sorry for not inserting the full log. After looking into carton's internals, I was wondering why you are using the spec strings consisting of Module Name and version number joined with a tilde. Could pathname (like "T/TY/TYEMQ/Algorithm-Diff-1.1902.tar.gz") do the job without having to care about version strings?

@miyagawa
Copy link
Contributor

Because that's not how CPAN clients figure out dependencies. Because Carton relies on cpanminus to resolve dependencies (at least for now) we have to use module and versions, then map them into pathname in 02packages.txt file internally (inside .carton directory)

jperkin pushed a commit to TritonDataCenter/pkgsrc-legacy that referenced this issue Dec 9, 2013
Based on PR pkg/46549 by Wen Heping(myself)

Upstream changes:
1.5017 Wed Jul 18 08:28:49 PDT 2012
   [Improvements]
      - Ignore tarballs whose first entry is "./" (doy) #184

1.5016 Tue Jul 17 12:00:57 PDT 2012
   [Improvements]
      - Added Module/CPANfile.pm to the fatlib. This allows bootstrapping dependencies
        with cpanm --installdeps on Heroku etc.

1.5015 Sun Jun 24 15:34:57 PDT 2012
   [Improvements]
      - Improved Makefile.PL to include bugtracker info (Ben Bullock)
      - Fixed some merge mess with devel

1.5014 Tue Jun 12 18:27:02 PDT 2012
   [Improvements]
      - Make sure 'f' flag becomes the last for some tar versions (mst, aaronsw)
      - Fixed warnings on perl 5.17+ (rjbs)
      - Fix local::lib error message (berekuk)

1.5013 Sat May 12 06:15:44 EEST 2012
   [Bug Fixes]
      - Fixed --cascade-search to seach for missing modules, which was broken by #150

1.5012 Fri May 11 05:47:56 CEST 2012
   [Improvements]
      - Change the behavior of --mirror-index so that it won't fallback to CPAN mirrors #150 (thaljef)
      - Support v-strings in versions specified install [perl-carton/carton#48]

1.5011 Thu Apr 12 18:57:06 JST 2012
   [Improvements]
      - Point default mirror to www.cpan.org #148

1.5010 Sat Mar 31 12:59:52 CEST 2012
   [New Feature]
      - Implemented --test-only option #40

1.5009 Fri Mar 30 18:44:12 CEST 2012
   [Bug Fixes]
      - Fixed a bug where URL containing ~ (childe) fails to install #134
      - Fixed a bug where install.json contains bogus data when you specify dist paths

1.5008 Sat Mar 17 18:19:57 PDT 2012
   [Bug Fixes]
      - Fied a bug where `cpanm Foo` doesn't properly activate local::lib during installs
        when you don't have write permissions #143 (goodel, ash)
      - Improved the warning message when you don't have write permisisons #145 (ash)
charsbar pushed a commit to charsbar/cpanminus that referenced this issue Feb 27, 2014
charsbar pushed a commit to charsbar/cpanminus that referenced this issue Feb 27, 2014
Changelog diff is:

diff --git a/Changes b/Changes
index cbe2ffb..e475125 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,10 @@
 See http://github.com/miyagawa/cpanminus/ for the latest development.

+1.5012 Fri May 11 05:47:56 CEST 2012
+   [Improvements]
+      - Change the behavior of --mirror-index so that it won't fallback to CPAN mirrors miyagawa#150 (thaljef)
+      - Support v-strings in versions specified install [perl-carton/carton#48]
+
 1.5011 Thu Apr 12 18:57:06 JST 2012
    [Improvements]
       - Point default mirror to www.cpan.org miyagawa#148
jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Oct 11, 2014
Based on PR pkg/46549 by Wen Heping(myself)

Upstream changes:
1.5017 Wed Jul 18 08:28:49 PDT 2012
   [Improvements]
      - Ignore tarballs whose first entry is "./" (doy) #184

1.5016 Tue Jul 17 12:00:57 PDT 2012
   [Improvements]
      - Added Module/CPANfile.pm to the fatlib. This allows bootstrapping dependencies
        with cpanm --installdeps on Heroku etc.

1.5015 Sun Jun 24 15:34:57 PDT 2012
   [Improvements]
      - Improved Makefile.PL to include bugtracker info (Ben Bullock)
      - Fixed some merge mess with devel

1.5014 Tue Jun 12 18:27:02 PDT 2012
   [Improvements]
      - Make sure 'f' flag becomes the last for some tar versions (mst, aaronsw)
      - Fixed warnings on perl 5.17+ (rjbs)
      - Fix local::lib error message (berekuk)

1.5013 Sat May 12 06:15:44 EEST 2012
   [Bug Fixes]
      - Fixed --cascade-search to seach for missing modules, which was broken by #150

1.5012 Fri May 11 05:47:56 CEST 2012
   [Improvements]
      - Change the behavior of --mirror-index so that it won't fallback to CPAN mirrors #150 (thaljef)
      - Support v-strings in versions specified install [perl-carton/carton#48]

1.5011 Thu Apr 12 18:57:06 JST 2012
   [Improvements]
      - Point default mirror to www.cpan.org #148

1.5010 Sat Mar 31 12:59:52 CEST 2012
   [New Feature]
      - Implemented --test-only option #40

1.5009 Fri Mar 30 18:44:12 CEST 2012
   [Bug Fixes]
      - Fixed a bug where URL containing ~ (childe) fails to install #134
      - Fixed a bug where install.json contains bogus data when you specify dist paths

1.5008 Sat Mar 17 18:19:57 PDT 2012
   [Bug Fixes]
      - Fied a bug where `cpanm Foo` doesn't properly activate local::lib during installs
        when you don't have write permissions #143 (goodel, ash)
      - Improved the warning message when you don't have write permisisons #145 (ash)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants