Skip to content

Allow 'haxelib local' to take web url #105

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

Closed
hughsando opened this issue Nov 24, 2013 · 15 comments
Closed

Allow 'haxelib local' to take web url #105

hughsando opened this issue Nov 24, 2013 · 15 comments
Assignees
Milestone

Comments

@hughsando
Copy link
Member

It would be nice to be able to point haxelib at a "wget" url as well as a local file.

@Simn
Copy link
Member

Simn commented Nov 27, 2013

haxelib local accepting web URLs is a bit of an oxymoron.

What is the use-case here?

@hughsando
Copy link
Member Author

True enough about the oxymoron.
The use case is that I want to provide nightly builds on the web - but not
to go out generally if someone wants just tries an "update". Perhaps there
is a better way. It is just to save some steps:
wget http://blah.zip
haxelib local blah.zip
rm blah.zip

vs:

haxelib url http://blah.zip

Hugh

On Wed, Nov 27, 2013 at 7:42 PM, Simon Krajewski
[email protected]:

haxelib local accepting web URLs is a bit of an oxymoron.

What is the use-case here?


Reply to this email directly or view it on GitHubhttps://github.com//issues/105#issuecomment-29378213
.

@Simn
Copy link
Member

Simn commented Nov 27, 2013

haxelib url would be fine with me.

@hughsando
Copy link
Member Author

While you are at it - got a bug on haxelib that reports:

Library nme version dev does not have a neko dll for your system

This comes from
haxe -cp "/usr/lib/haxe/lib/haxelib_client/3,1,0-rc,3/" -D haxelib_client
--run tools.haxelib.Main $@

returning "linux" for Sys.systemName() on a mac. I presume this is from
the haxe interpret code.
Caused me some grief (because I removed the linux directory independently,
which confused me).
Nice to have source code though.

Would you like me to report a separate bug?

Hugh

On Wed, Nov 27, 2013 at 8:45 PM, Simon Krajewski
[email protected]:

haxelib url would be fine with me.


Reply to this email directly or view it on GitHubhttps://github.com//issues/105#issuecomment-29381592
.

@Simn
Copy link
Member

Simn commented Nov 27, 2013

Yes, please file a separate bug.

@jasononeil
Copy link
Contributor

Would it be too much overloading to use "haxelib install"?

Currently we have

haxelib install markdown # from haxelib
haxelib install markdown.zip # local
haxelib install build.hxml # all libs in hxml
haxelib install all # all hxml files in CWD

And correspondingly rules that a library name cannot be "all", or end in
".zip" or ".html". So we could add:

haxelib install https://github.com/haxetink/tink_core/archive/master.zip

And library names starting with "http://" or "https://" are disallowed.
(We may even have a regex preventing these names already?)

Cheers for the massive Haxelib cleanup!
On 27 Nov 2013 20:45, "Simon Krajewski" [email protected] wrote:

haxelib url would be fine with me.


Reply to this email directly or view it on GitHubhttps://github.com//issues/105#issuecomment-29381592
.

@hughsando
Copy link
Member Author

Yes, I think a regex would be much better, and have a single "haxelib install file.zip|name[version]|http"

Hugh

@Simn
Copy link
Member

Simn commented Nov 28, 2013

I didn't know that haxelib install is so versatile already. In that case I agree we can extend it to check for http(s) prefix.

@ncannasse
Copy link
Member

While we're at it we could also handle git:// svn:// and http://github.com/ncannasse/h3d as well

@Simn
Copy link
Member

Simn commented Nov 28, 2013

@jasononeil: To clarify, all these haxelib install checks you mention are available in the Haxelib version that comes with 3.0.0? I have to get this right in the new documentation.

@back2dos
Copy link
Member

@Simn yep, those are all supported:

function install() {

@Simn
Copy link
Member

Simn commented Nov 28, 2013

I understand that they are supported, I need to know if they were supported in the Haxelib version that came with 3.0.0.

@back2dos back2dos reopened this Nov 28, 2013
@back2dos
Copy link
Member

Yes, that's what I meant to say. That's the implementation for Haxelib 3.0.0.

@back2dos back2dos self-assigned this May 28, 2014
@back2dos back2dos modified the milestones: 3.2.0, 3.2.0-rc.1 Mar 9, 2015
@alebianco
Copy link

@back2dos
just for confirmation, the syntax haxelib install http://host/archive.zip would not work in the current implementation, correct?
I think that would be enough to solve the issue

@nadako
Copy link
Member

nadako commented Dec 15, 2015

I think we can merge this issue into #238, since we're discussing npm-like url support there.

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

7 participants