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

ClassPath isn't a Haxelib! #100

Closed
boozook opened this issue Jan 11, 2015 · 7 comments
Closed

ClassPath isn't a Haxelib! #100

boozook opened this issue Jan 11, 2015 · 7 comments

Comments

@boozook
Copy link

boozook commented Jan 11, 2015

Works on a latest build.

After indexation of opened project we can see two libs automatically added as dependencies. But it's not from -lib ... from build.hxml ! This is -cp Classpaths.
screen shot 2015-01-11 at 17 29 08
screen shot 2015-01-11 at 17 38 02
screen shot 2015-01-11 at 17 38 38

Linked hxml:

-cp src
-cp tests
-main Main
-lib signal
-dce full
--each
-x out/tests/main

TIR: STB-7318

@as3boyan
Copy link
Contributor

Ok, thanks, src should be resolved relative to project root directory, but first, we need to make sure that it's not already added.

@EBatTiVo
Copy link
Contributor

EBatTiVo commented Feb 4, 2015

There is code in place to deal with the "already added" case, though I guess it could be broken. In this case, we should NOT be adding libs that come from the classpath at all. As I remember, all of the libraries that we add come from calling the haxelib command. We don't add them from any other configuration file. Could haxelib be presenting us libraries from the classpath?

@boozook
Copy link
Author

boozook commented Jul 17, 2015

Again. Release 0.9.5 on Idea 14 Community Edition (14.1.4)

hx-plugin-cp-bug mov opt

It happening after reload project with .hxml with -cp.

@as3boyan
Copy link
Contributor

It seems like it doesn't checks project paths.

@eliasku
Copy link
Contributor

eliasku commented Feb 19, 2016

+1
Very critical!

@eliasku
Copy link
Contributor

eliasku commented Sep 29, 2016

@as3boyan @EBatTiVo ClassPath should be not added as EXTERNAL libraries!!! ClassPath should be marked by user in module configuration. Adding -cp as libraries is wrong behaviour.

@EricBishton
Copy link
Member

Fixed a while ago and no further action appears to be required. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants