[Bug 894725] Review Request: uglify-js - JavaScript parser, mangler/compressor and beautifier toolkit

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=894725

--- Comment #8 from T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> ---
(In reply to comment #6)
> How are you going to bootstrap the uglify-js? There is no preparation in any
> its build time dependencies. I installed manually all its dependencies using
> rpm --nodpes, but that is not an option for Koji build.

Its BuildRequires need to go into Fedora first.  nodejs-source-map is the only
thing that needs any bootstrapping logic, and that just prevents the browser
version from being built until uglify-js in in the distribution.

You can drop the BuildRequires and %check if you want it to just build without
the deps, but we need the deps at runtime anyway so they need to get done no
matter what we do, so I didn't bother leaving them out.

(In reply to comment #7)
> Btw the build fails for me in %install section:
> 
> + cp -pr bin lib package.json uglify-js.js
> /builddir/build/BUILDROOT/uglify-js-2.2.3-3.fc19.x86_64/usr/share/uglify-js/
> cp: cannot stat 'uglify-js.js': No such file or directory

Oops, sorry I broke it in the updated version.  Fixed.

Spec: http://patches.fedorapeople.org/npm/uglify-js.spec
SRPM: http://patches.fedorapeople.org/npm/uglify-js-2.2.3-4.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=UuilN1SeTs&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]