Bug 614297

Summary: Review Request: meegotouch-theme - Theme for Meego Touch based applications
Product: [Fedora] Fedora Reporter: Chen Lei <supercyper1>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED NOTABUG QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: rawhideCC: fedora-package-review, jreznik, otto.liljalaakso
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-05-22 00:45:23 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 632051    
Bug Blocks: 201449    

Description Chen Lei 2010-07-14 03:40:56 UTC
Description:
Theme required by any MeeGo Touch library based application. Contains the base
theme files and two alternative themes with a different look.
SPEC:http://dl.dropbox.com/u/1338197/1/meegotouch-theme.spec
SRPM:http://dl.dropbox.com/u/1338197/1/meegotouch-theme-0.20.14-1.fc13.src.rpm

Comment 1 Chen Lei 2010-07-14 03:44:16 UTC
Note: This package should be reviewed before libmeegotou even it depends on it, because libmeegotouch also depends on meegotouch-theme.

Comment 2 Chen Lei 2010-07-14 03:45:04 UTC
s/it depends on it/it depends on libmeegotouch

Comment 3 Jaroslav Reznik 2010-09-06 13:43:33 UTC
Could you send the link to libmeegotouch review?

Comment 4 Jaroslav Reznik 2010-09-06 13:55:02 UTC
(In reply to comment #1)
> Note: This package should be reviewed before libmeegotou even it depends on it,
> because libmeegotouch also depends on meegotouch-theme.

Sounds like circular dependency. On the other, you should not Require libmeegoutouch explicitely.

Comment 5 Chen Lei 2010-09-06 14:45:03 UTC
(In reply to comment #3)
> Could you send the link to libmeegotouch review?

I still waiting a review for libqttracker which is a dependency for libmeegotouch.

See
https://bugzilla.redhat.com/show_bug.cgi?id=614075

(In reply to comment #4)
> (In reply to comment #1)
> > Note: This package should be reviewed before libmeegotou even it depends on it,
> > because libmeegotouch also depends on meegotouch-theme.
> Sounds like circular dependency. On the other, you should not Require
> libmeegoutouch explicitely.

I think yum is smart enough to solve circular dependency :)

Comment 6 Jaroslav Reznik 2010-09-06 14:53:49 UTC
But you still should not Require it explicitly? Or is there any reason to do it so?

Comment 7 Chen Lei 2010-09-06 15:19:09 UTC
(In reply to comment #6)
> But you still should not Require it explicitly? Or is there any reason to do it
> so?

The upstream rpm depends on libmeegotouch, I'll take a look at it if this dependency is necessary.

Comment 8 Jaroslav Reznik 2010-09-07 08:56:57 UTC
Fedora has different packaging guidelines than MeeGo.

Comment 9 Chen Lei 2010-09-07 10:22:24 UTC
(In reply to comment #8)
> Fedora has different packaging guidelines than MeeGo.

It seems meegotouch-theme depends on libmeegotouch because of %{_sysconfdir}/xdg/autostart/mthemedaemon.desktop.  Actually, there are many circular dependency in fedora(e.g. phonon and phonon-backend).

Comment 11 Package Review 2020-07-10 00:45:37 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 12 Package Review 2020-11-13 00:45:27 UTC
This is an automatic action taken by review-stats script.

The ticket reviewer failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we reset the status and the assignee of this ticket.

Comment 13 Otto Liljalaakso 2021-04-21 07:11:53 UTC
This review request is very old. If so, I can do the review, please provide new urls for the specfile and srpm, the old ones are stale. If you are not interested anymore, either close this issue or do nothing, in which case automation should close this issue in one month.

Comment 14 Package Review 2021-05-22 00:45:23 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.