Giter Club home page Giter Club logo

Comments (19)

jolexa avatar jolexa commented on May 22, 2024

It looks like the recipe is already doing some trickery for smartos, should be trivial to add a line for rhel,centos

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

Found a quick-fix: jpackage repository provides a package named 'tomcat7' :)

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

@shaycohen is jpackage still supported? I cannot seem to find info about jpackage on centos 6.

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

@joerocklin:
http://wiki.centos.org/AdditionalResources/Repositories
On Jun 23, 2014 5:57 PM, "Joe Rocklin" [email protected] wrote:

@shaycohen https://github.com/shaycohen is jpackage still supported? I
cannot seem to find info about jpackage on centos 6.

Reply to this email directly or view it on GitHub
#74 (comment)
.

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

@shaycohen right, and the last update to the jpackage site and repo list is from 2009 referencing java 5 (if I'm reading things correctly). That doesn't exactly strike me as 'up to date'.

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

@joerockin, you are correct. Although I did not manage to find any Java/JRE version on JPackage 6.0, I noticed (while looking) that the published Tomcat version is 7.0.39 (over a year ago).

I think this issue can be resolved pretty quickly. Actually the main reason for not fixing the code (probably adding a search for tomcat-7, tomcat7, tomcat_7, etc; instead of walking through a list of known distributions) is to be able to maintain the cookbook intact.
I think I'll just edit my version of the fix and commit it sometime soon..

Thanks for being alert/noticing :)

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

@shaycohen I too started on a tomcat7/centos6 update for this cookbook, though I'm not to a place where I really have it working just yet (I'm not really a tomcat person, so it's probably something simple that I'm missing). If you've got something close I'd be happy to help test where I can, though I'll repeat: I'm not a tomcat person, so the best testing I'll be able to provide is 'does it install'.

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

Sure, I'll upload my progress .. did you upload what you have ?

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

I just pushed it to https://github.com/joerocklin/tomcat/tree/tomcat7-centos6.

I went the route of changing the package installation method to use attributes instead internal variables. I found I was basically creating a new attributes section in the recipe just for tomcat7/centos though, and that felt a bit weird to me.

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

I haven't had any 'vi time' yet (probably only tomorrow), but I would probably:

  • Seek a better way than "depends 'yum-epel ..." on metadata, as this will probably break the recipe on anything other than a CentOS or a Redhat
  • remove base_instance duplicate declaration on providers/instance and stick with the original one (you are dealing with this on the attributes later on

You made some great changes. This recipe seems to be very statically written initially.

thanks

from tomcat.

jolexa avatar jolexa commented on May 22, 2024

Doesn't PR #47 resolve this issue too? #47

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

@shaycohen I'm pretty sure that the yum-epel recipe will only get included if chef is running on CentOS and the attribute is set to install tomcat7. This should probably be expanded to the rhel platform_family, but I was doing focused testing. I would rather install from a package repo (that's getting updates) than switch to a tarball install for a specific platform/version combo.

@jolexa #47 sets up the package settings to be able to handle this better, and I wish it had been pulled in (I tried to use it as a starting point, but it doesn't merge at this point). It does not support tomcat7 on centos6 though, because the packaging of tomcat7 from EPEL drops the seven and just uses 'tomcat'.

from tomcat.

joerocklin avatar joerocklin commented on May 22, 2024

Looking into it a bit more, the EPEL rpm for tomcat pulls in redhat-lsb, and therefore pulls in a ton more than what is necessary to run tomcat. Not sure why that decision was made...
@shaycohen have you found a good way to put tomcat7 on a centos 6 box w/o EPEL?

from tomcat.

shaycohen avatar shaycohen commented on May 22, 2024

Got a bit of a break from that issue for a while (monitoring stuff); I
think my path will be to use tomcat7 and jdk distributed as a ready-to-use
directory.
I think that trying to make this work in the standard chef approach will be
much more fragile and will only be beneficial for those who doesn't know
for sure the distribution they are using.

On Mon, Jul 7, 2014 at 8:46 PM, Joe Rocklin [email protected]
wrote:

Looking into it a bit more, the EPEL rpm for tomcat pulls in redhat-lsb,
and therefore pulls in a ton more than what is necessary to run tomcat. Not
sure why that decision was made...
@shaycohen https://github.com/shaycohen have you found a good way to
put tomcat7 on a centos 6 box w/o EPEL?

Reply to this email directly or view it on GitHub
#74 (comment)
.

from tomcat.

cwebberOps avatar cwebberOps commented on May 22, 2024

Notes from hacking session:

  • depends on yum-epel but dont require recipe, use like apt
  • default recipe needs to be updated to handle the differences between centos5 vs centos6

from tomcat.

joshskains avatar joshskains commented on May 22, 2024

So is this worth waiting for or should I make a hack? I have CentOS 6.5 and need tomcat7. I have EPEL enabled, but the package name is of course breaking the install. I am curious if an official fix will be made or do I need to figure something out? Thanks!

from tomcat.

damacus avatar damacus commented on May 22, 2024

@joshskains if you can provide/propose a fix that takes into account the CentOS 5 vs 6 and submit a PR that'd be great.
Even if it's just to get us talking about it again.

from tomcat.

jperry avatar jperry commented on May 22, 2024

++ on this. Any progress or update?

from tomcat.

lock avatar lock commented on May 22, 2024

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

from tomcat.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.