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

name agrument not passed correctly #265

Closed
hstejas opened this issue Sep 24, 2017 · 5 comments
Closed

name agrument not passed correctly #265

hstejas opened this issue Sep 24, 2017 · 5 comments

Comments

@hstejas
Copy link

hstejas commented Sep 24, 2017

This is Rakudo version 2017.09 built on MoarVM version 2017.09.1
implementing Perl 6.c.

Info for: zef
Identity: zef:ver<0.1.29>:authgithub:ugexe

OS: Ubuntu 17.04

This is on a fresh install of perl6 and bailador (tried manually building rakudo and using rakudobrew)

bailador --name=test new

gives this error

 --name=Project-Name is a required parameter

whereas directly calling

rakudo/install/share/perl6/site/resources/F3147CDB5FA175F21CCAC05B6B29A24EF5EAED7B --name=test new

works fine

This diff points to the problem (./rakudo/install/share/perl6/site/bin/bailador)

@@ -1,4 +1,5 @@
 #!/usr/bin/env perl6
 sub MAIN(:$name is copy, :$auth, :$ver, *@, *%) {
-    CompUnit::RepositoryRegistry.run-script("bailador", :dist-name<Bailador>, :$name, :$auth, :$ver);
+       my $x = "--name=" ~ $name;
+    CompUnit::RepositoryRegistry.run-script("bailador", :dist-name<Bailador>, :$x, :$auth, :$ver);
 }

not sure if it is a zef issue or a bailador issue

@ufobat
Copy link
Member

ufobat commented Sep 30, 2017

this script is not part of Bailador.

@szabgab
Copy link
Member

szabgab commented Sep 30, 2017

Isn't this the script that is generated by zef when we install Bailador?

@ufobat
Copy link
Member

ufobat commented Sep 30, 2017

@ufobat
Copy link
Member

ufobat commented Sep 30, 2017

=> we shoudn't use --name as a paremeter.
no --name, no --auth, no --ver

@ufobat
Copy link
Member

ufobat commented Oct 1, 2017

will be fixed in 0.0.13

@ufobat ufobat closed this as completed Oct 1, 2017
ufobat added a commit that referenced this issue Oct 1, 2017
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

3 participants