Bug 1196713 - thermostat depends on jetty, but not on thermostat-webapp
Summary: thermostat depends on jetty, but not on thermostat-webapp
Alias: None
Product: Fedora
Classification: Fedora
Component: thermostat
Version: 22
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Severin Gehwolf
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2015-02-26 15:09 UTC by Severin Gehwolf
Modified: 2015-03-31 22:01 UTC (History)
4 users (show)

Fixed In Version: thermostat-1.2.2-5.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2015-03-31 22:01:28 UTC
Type: Bug

Attachments (Terms of Use)

Description Severin Gehwolf 2015-02-26 15:09:39 UTC
Description of problem:
When installing thermostat in F22/rawhide it drags in jetty dependencies due to "thermostat web-storage-service". However, there is no guarantee that the webapp itself (which "thermostat web-storage-service" needs) is installed.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1. yum erase thermostat\*
2. yum install thermostat

Actual results:
Step two drags in jetty deps.

Expected results:
No jetty deps get installed. The web-storage-service plugin should not get installed.

Additional info:
web-storage-service should be moved to package thermostat-webapp or into a separate sub-package which then depends on thermostat-webapp. The most clean solution would be to only have the webapp in thermostat-webapp (with no tomcat deps). Then add other sub-packages which either drag in tomcat or jetty. Whatever the user wants.

There is one more problem: thermostat-setup assumes the webapp is installed (for changing WEB-INF/web.xml) which might not be the case.

Comment 1 Severin Gehwolf 2015-03-24 17:35:59 UTC
Fixed with thermostat-1.2.2-5.fc22:

Comment 2 Severin Gehwolf 2015-03-24 17:36:37 UTC
web-storage-service is now part of thermostat-webapp.

Comment 3 Fedora Update System 2015-03-24 17:58:46 UTC
thermostat-1.2.2-5.fc22 has been submitted as an update for Fedora 22.

Comment 4 Fedora Update System 2015-03-26 22:03:17 UTC
Package thermostat-1.2.2-5.fc22:
* should fix your issue,
* was pushed to the Fedora 22 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing thermostat-1.2.2-5.fc22'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).

Comment 5 Fedora Update System 2015-03-31 22:01:28 UTC
thermostat-1.2.2-5.fc22 has been pushed to the Fedora 22 stable repository.  If problems still persist, please make note of it in this bug report.

Note You need to log in before you can comment on or make changes to this bug.