Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: pass (asf.osuosl.org: domain of alexey.a.petrenko@gmail.com
 designates 66.249.82.225 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=hqZoz9MVVN5ncP8FSXNN5SPsAcLKqaKCFKhBTSqKFjCxA2CK5agRiwIz/xiP1fBQGFAjB2hxfZIweR8zXkxaEhJvfYFT9T6hPusdriwkjjxGztYzKplq6ChVTVG5Mq7gJhEkWumnOyTyMG94nsG9NqyCv+dnOwmAFGPYJu2RJf8=
Message-ID: <c3755b3a0609062344k2d309d61x8b3bdc6f6e7c8770@mail.gmail.com>
Date: Thu, 7 Sep 2006 10:44:49 +0400
From: "Alexey Petrenko" <alexey.a.petrenko@gmail.com>
To: harmony-dev@incubator.apache.org
Subject: Re: [vote] HARMONY-1363 - DRLVM fixes and additions
In-Reply-To: <f944d83d0609062330o7ff1cbadlb6e38f26028cb6d9@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <44FFA109.7080600@pobox.com> <44FFA12D.2070307@pobox.com>
	 <f944d83d0609062151g3694501cr6566eb0b0023058f@mail.gmail.com>
	 <c3755b3a0609062238y1e2234c2ta7cd33590c8a77bb@mail.gmail.com>
	 <f944d83d0609062330o7ff1cbadlb6e38f26028cb6d9@mail.gmail.com>

I've talked about a small patch for this exact issue of course.

SY, Alexey

2006/9/7, Alexey Varlamov <alexey.v.varlamov@gmail.com>:
> Alexey,
>
> This conflict is trivial enough and was caused by recent Geir's
> changes, see the parallel thread about launcher.
> I don't think we need to redone the whole patch for this issue -
> either commiter will resolve it by hand or minor patch for this
> particular change can be made anew.
>
> --
> Regards,
> Alexey
>
>
> 2006/9/7, Alexey Petrenko <alexey.a.petrenko@gmail.com>:
> > Guys,
> >
> > I've tried a patch.
> > Patch failed in ./vm/em/src/DrlEMImpl.cpp. On Linux and Windows.
> > Here is a rej file:
> > === cut ===
> > ***************
> > *** 151,165 ****
> >  //_____________________________________________________________________
> >  // Reading and parsing configuration
> >
> >
> > - std::string buildDefaultLibPath(const std::string& dll_name) {
> > -     std::string library_path = vm_get_property_value("vm.boot.library.path");
> >  #ifdef PLATFORM_NT
> > -     std::string fullPath = library_path + "\\"+ dll_name+".dll";
> > - #else
> > -     std::string fullPath = library_path + "/lib" + dll_name + ".so";
> >  #endif
> > -     return fullPath;
> >  }
> >
> >  static std::string getParam(const std::string& config, const
> > std::string& name) {
> > --- 167,198 ----
> >  //_____________________________________________________________________
> >  // Reading and parsing configuration
> >
> > + static bool endsWith(const std::string& str, const std::string& suffix) {
> > +     if (str.length() < suffix.length()) {
> > +         return false;
> > +     }
> > +     return std::equal(suffix.rbegin(), suffix.rend(), str.rbegin());
> > + }
> >
> > + std::string prepareLibPath(const std::string& origPath) {
> >  #ifdef PLATFORM_NT
> > +     std::string separator("\\"), libPrefix(""), libSuffix(".dll");
> > + #else
> > +     std::string separator("/"), libPrefix("lib"), libSuffix(".so");
> >  #endif
> > +
> > +     std::string path = origPath;
> > +     if (path.find('/') == path.npos && path.find('\\') == path.npos ) {
> > +         std::string dir = vm_get_property_value("vm.boot.library.path");
> > +         if (libPrefix.length() > 0 && !startsWith(path, libPrefix)) {
> > +             path = libPrefix + path;
> > +         }
> > +         path = dir + separator + path;
> > +     }
> > +     if (!endsWith(path, libSuffix)) {
> > +         path+=libSuffix;
> > +     }
> > +     return path;
> >  }
> >
> >  static std::string getParam(const std::string& config, const
> > std::string& name) {
> > === cut ===
> >
> > As a result build fails.
> >
> > Salikh, it seems that you need to provide additional patch.
> >
> > SY, Alexey
> >
> > 2006/9/7, Alexey Varlamov <alexey.v.varlamov@gmail.com>:
> > > +1
> > >
> > > 2006/9/7, Geir Magnusson Jr. <geir@pobox.com>:
> > > > +1
> > > >
> > > > Geir Magnusson Jr. wrote:
> > > > > All is in order and in SVN for Harmony-1225 wrt BCC and ACQ.  I think
> > > > > that this an important patch so we can get better 1.5 support et al.
> > > > >
> > > > > Please vote to accept or reject this set of patches and fixes into the
> > > > > Apache Harmony class library :
> > > > >
> > > > > [ ] + 1 Accept
> > > > > [ ] -1 Reject  (provide reason below)
> > > > >
> > > > > Lets let this run a minimum of 3 days unless a) someone states they need
> > > > > more time or b) we get all committer votes before then.
> > > > >
> > > > > geir
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > ---------------------------------------------------------------------
> > > > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > > > >
> > > >
> > > > ---------------------------------------------------------------------
> > > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > > >
> > > >
> > >
> > > ---------------------------------------------------------------------
> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > >
> > >
> >
> >
> > --
> > Alexey A. Petrenko
> > Intel Middleware Products Division
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>


-- 
Alexey A. Petrenko
Intel Middleware Products Division

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org