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

rebar2 get-deps fails #24

Open
mrbegood opened this issue Dec 16, 2016 · 6 comments
Open

rebar2 get-deps fails #24

mrbegood opened this issue Dec 16, 2016 · 6 comments

Comments

@mrbegood
Copy link

mrbegood commented Dec 16, 2016

Hi, this commit 964310d breaks dependency compilation with rebar2

rebar version
rebar 2.6.4 17 20160926_070944 git 2.6.4-4-g576e121  

rebar get-deps

...

==> lager_syslog (get-deps)
Pulling syslog from {git,"git://github.com/Vagabond/erlang-syslog",
                         {branch,"master"}}
Cloning into 'syslog'...
Uncaught error in rebar_core: {'EXIT',
                               {function_clause,
                                [{code,which,
                                  [{pc,
                                    {git,
                                     "[email protected]:blt/port_compiler.git",
                                     {branch,"master"}}}],
                                  [{file,"code.erl"},{line,719}]},
                                 {rebar_core,'-plugin_modules/3-lc$^0/1-0-',
                                  1,
                                  [{file,"src/rebar_core.erl"},{line,573}]},
                                 {rebar_core,plugin_modules,3,
                                  [{file,"src/rebar_core.erl"},{line,573}]},
                                 {rebar_core,process_dir1,7,
                                  [{file,"src/rebar_core.erl"},{line,244}]},
                                 {rebar_core,process_each,5,
                                  [{file,"src/rebar_core.erl"},{line,351}]},
                                 {rebar_core,process_dir1,7,
                                  [{file,"src/rebar_core.erl"},{line,271}]},
                                 {rebar_core,process_each,5,
                                  [{file,"src/rebar_core.erl"},{line,351}]},

To re-produce problem you can try to build this app
https://github.com/esl/MongooseIM

mkdir -p /tmp/mongooseim \
&& cd /tmp/mongooseim \
&& git clone [email protected]:esl/MongooseIM.git . \
&& ./tools/configure \
&& make rel

When adding dependency to this commit e24c9ee in rebar.config

{syslog, ".*", {git, "git://github.com/Vagabond/erlang-syslog", "e24c9ee"}}

rebar get-deps works good.

@Vagabond
Copy link
Owner

cc @binarytemple-bet365

@binarytemple-external
Copy link
Contributor

@mrbegood - The project has moved to rebar3, so master will not compile with rebar, as the dependencies format is different (rebar is deprecated in any case).

However I don't think that is the problem here. I initially suspected that MongooseIM might have a dependency upon erlang-syslog master but it doesn't seem to be so:

Examining MongooseIM master:

MongooseIM has the following dependency on lager_syslog:

 {lager_syslog, ".*", {git, "git://github.com/basho/lager_syslog.git", "3.0.3"}},

lager_syslog in turn has the followowing dependency on erlang-syslog:

{'syslog', "1.0.*", {git, "https://github.com/basho/erlang-syslog", {tag, "1.0.3"}}}

So to test (I can't actually build MongooseIM due to it's fast_tls dependency choking on EC support):

% git clone https://github.com/basho/erlang-syslog     
Cloning into 'erlang-syslog'...                                                  
remote: Counting objects: 177, done.                                             
remote: Total 177 (delta 0), reused 0 (delta 0), pack-reused 177                 
Receiving objects: 100% (177/177), 210.40 KiB | 0 bytes/s, done.                 
Resolving deltas: 100% (75/75), done.                                            
% cd erlang-syslog                                     

% git checkout 1.0.3                     
Note: checking out '1.0.3'.                                                      

etc etc etc...                                                                                 
                                                                                 
HEAD is now at e24c9ee... Roll 1.0.3                                             
                    
% ./rebar compile                          
==> erlang-syslog (compile)                                                      
Compiled src/syslog_app.erl                                                      
Compiled src/syslog_sup.erl                                                      
Compiled src/syslog.erl                                                          
Compiling c_src/syslog_drv.c                                                     
%                                        

Can you perform a fresh checkout of MongooseIM and try again please?

@binarytemple-external
Copy link
Contributor

binarytemple-external commented Dec 19, 2016

@mrbegood Has your issue been fixed by this @michalwski P/R ?

If so, we can close this ticket.

@binarytemple-external
Copy link
Contributor

@mrbegood - in any event we have this P/R from @vavaka which will make this project work with rebar and rebar3 - #25

@binarytemple-external
Copy link
Contributor

@mrbegood merged to master, please test and close.

@tsloughter
Copy link
Contributor

rebar2 is no longer supported. This issue should be closed.

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

4 participants