Bug 858027 - Review Request: jquery-ui - JavaScript library of interface components based on jQuery
Summary: Review Request: jquery-ui - JavaScript library of interface components based ...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 815018 977123 js-jquery1 1115672 1115673
Blocks: FE-DEADREVIEW 603937 1154750
TreeView+ depends on / blocked
 
Reported: 2012-09-17 19:54 UTC by Gregor Tätzner
Modified: 2020-07-11 00:46 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-07-11 00:46:40 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Gregor Tätzner 2012-09-17 19:54:38 UTC
Spec URL: http://brummbq.fedorapeople.org/jquery-ui.spec
SRPM URL: http://brummbq.fedorapeople.org/jquery-ui-1.8.23-1.fc17.src.rpm
Description: 
jQuery UI provides abstractions for low-level interaction and animation,
advanced effects and high-level, themeable widgets, built on top of the
jQuery JavaScript Library, that you can use to build highly interactive
web applications.
Fedora Account System Username: brummbq

Comment 1 T.C. Hollingsworth 2013-01-17 09:10:27 UTC
Node.js is available for Fedora 18 and rawhide and should no longer block this package.

Comment 2 Felix Schwarz 2013-02-19 10:00:55 UTC
Why is nodejs a dependency for jQuery UI?

Comment 3 Jens Petersen 2013-09-19 08:41:39 UTC
I think this needs updating for:

https://fedoraproject.org/wiki/Changes/Web_Assets
https://fedoraproject.org/wiki/Packaging:JavaScript

Comment 4 Jens Petersen 2013-09-19 08:45:36 UTC
(In reply to Felix Schwarz from comment #2)
> Why is nodejs a dependency for jQuery UI?

A build dependency - see the comments in the spec file:

%build
# since grunt/npm is not available in fedora, it does not make much sense
# to build jqueryui from source

For grunt

https://bugzilla.redhat.com/buglist.cgi?component=Package%20Review&list_id=1723739&short_desc=grunt&short_desc_type=substring

Comment 5 Jens Petersen 2013-09-19 09:03:00 UTC
(In reply to Jens Petersen from comment #4)
> (In reply to Felix Schwarz from comment #2)
> > Why is nodejs a dependency for jQuery UI?

Sorry, perhaps meant why nodejs and not npm review is listed as a dependent bug?
Maybe because the npm review was posted after this one.

Gregor, are you planning to update the package?

Comment 6 T.C. Hollingsworth 2013-09-19 12:13:31 UTC
(In reply to Jens Petersen from comment #5)
> Sorry, perhaps meant why nodejs and not npm review is listed as a dependent
> bug?
> Maybe because the npm review was posted after this one.

npm itself really isn't of much help, as it's entirely inappropriate to use it to install dependencies in spec files due to the bundled libraries policy.  

But Gregor couldn't possibly have known how the nodejs stuff was going to shape up when he did this originally, as we had almost abandoned all hope of ever getting it in at the time.  ;-)

Anyway, I'm marking jquery as blocking this.  Please help with some of the reviews blocking jQuery if you want to move this along quicker.

Comment 7 T.C. Hollingsworth 2013-09-19 12:25:38 UTC
devDependencies not yet packaged that are not also needed by jquery itself:
"grunt-contrib-qunit": "0.2.0",
"grunt-contrib-csslint": "0.1.1",
"grunt-contrib-cssmin": "0.4.2", #easyfix, ycssmin is already in
"grunt-html": "0.3.3",

Comment 8 Zbigniew Jędrzejewski-Szmek 2015-01-28 05:29:30 UTC
nodejs-grunt-contrib-csslint and nodejs-grunt-contrib-cssmin are in.

Comment 9 Dominik 'Rathann' Mierzejewski 2015-05-13 10:16:48 UTC
Any progress here? I need this to unbundle jquery-ui from mozilla-privacy-badger Firefox extension. I could review an updated package...

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

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the
NEEDINFO flag.

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at
https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group.

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.

Comment 11 Package Review 2020-07-11 00:46:40 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.


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