On Fri, Sep 23, 2016 at 12:51 AM, Jong Wook Kim <jongw...@nyu.edu> wrote:
> I wrote that section in the README.md in mind that the majority will > download the binary distribution which will contain the content in the > subdirectory, > > The section “Building from the source” specifies that the distribution > will be created in target/apache-s2graph-$version-incubating-bin. > > So I don’t think this is a blocker, but I agree that README.md can be even > more specific and it would be nice if start-s2graph.sh prints a helpful > message if it is launched in the source root. > No no, the mistake was completely on my side, I just wanted to provide feedback from outside of the project development. Of course this is not blocking the release, it's just a comment. > > On Sep 22, 2016, at 6:02 AM, Sergio Fernández <wik...@apache.org> wrote: > > > > On Thu, Sep 22, 2016 at 11:56 AM, Kim, Min-Seok <mskim....@gmail.com> > wrote: > >> > >> To Sergio, > >> > >> The commands will work on > >> `/path/to/incubator-s2graph/target/apache-s2graph-0.1.0-incubating-bin` > >> after `sbt package`. > >> > > > > Of course! > > > > Then the README needs to remark that to avoid people to get confused, > like > > it happened to me. > > > > Thanks! > > > > -- > > Sergio Fernández > > Partner Technology Manager > > Redlink GmbH > > m: +43 6602747925 > > e: sergio.fernan...@redlink.co > > w: http://redlink.co > > -- Sergio Fernández Partner Technology Manager Redlink GmbH m: +43 6602747925 e: sergio.fernan...@redlink.co w: http://redlink.co