On 10-Oct-2012, at 12:22 PM, Edison Su <[email protected]> wrote:
> Don't add javassit jar into deps rpm! It breaks rpm build. > Javassit is a dependence, included in centos 6.3. Again, if it's not in RHEL > Iso, need to turn on RHN. Check again all your commits. javassist is an apache license compliant package that is fetched by maven. We've a lot of packages that may be on one distro but not on other, hence we're bundling them in cloud-deps package. We can have a discussion on adding suffix cloud- on them to avoid possible install conflicts, but only post 4.0 > > Sent from my iPhone > > On Oct 9, 2012, at 11:03 PM, "Rohit Yadav" <[email protected]> wrote: > >> >> On 10-Oct-2012, at 11:24 AM, Rohit Yadav <[email protected]> wrote: >> >>> I'm maintaining the nonoss build patch which fails for the commit >>> c49f3beccfcd1257eca1ea06606fb55b3fdf5093 which removes javassist jar from >>> cloud-deps. >>> For this one has to either configure a CentOS repo and the dependency will >>> get downloaded, or install javassist manually. >>> >>> But this applies for a lot of other packages that may be downloaded from a >>> distro's repo such as axis, axiom etc. then cloud-deps may not have any use. >>> Since we're in code freeze now, I'm reverting let's do this discussion post >>> 4.0 and make any such changes. >> >> I followed the commits, with triple reverts, I held my revert back :( I'll >> ask QA to just install javassist manually now, since you would know better >> what you did around that, pl. see this issue. >> >> Regards.
