Bug 1823724 - Rename Review Request: yarnpkg - Fast, reliable, and secure dependency management
Summary: Rename Review Request: yarnpkg - Fast, reliable, and secure dependency manage...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-04-14 11:01 UTC by Zuzana Svetlikova
Modified: 2020-04-19 22:47 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-04-19 22:47:41 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Zuzana Svetlikova 2020-04-14 11:01:01 UTC
Spec URL: https://fedorapeople.org/~zvetlik/yarnpkg/yarnpkg.spec
SRPM URL: https://fedorapeople.org/~zvetlik/yarnpkg/yarnpkg-1.22.4-1.fc33.src.rpm
Description: Fast, reliable, and secure dependency management
Fedora Account System Username: zvetlik

As per https://src.fedoraproject.org/rpms/nodejs-yarn/pull-request/3

Comment 1 Neal Gompa 2020-04-14 12:07:32 UTC
Spec is essentially identical minus the subpackage thing, so LGTM.

PACKAGE APPROVED.

Comment 2 Miro Hrončok 2020-04-14 13:16:29 UTC
This is weird:

#%{nodejs_symlink_deps}

The macro will still expand. Is that intended?

Comment 3 Stephen Gallagher 2020-04-14 14:13:09 UTC
(In reply to Miro Hrončok from comment #2)
> This is weird:
> 
> #%{nodejs_symlink_deps}
> 
> The macro will still expand. Is that intended?

The macro happens to be a single line[1], so it will be commented-out, but you're right. This should be
`#%%{nodejs_symlink_deps}` for safety. Or removed entirely, of course. I suspect it's there because the standard template for Node packages would use it, but as yarn has no dependencies on other Fedora-packaged Node modules, it's unneeded.


[1]
```
%nodejs_sitelib %{_prefix}/lib/node_modules
%nodejs_symlink_deps %{_rpmconfigdir}/nodejs-symlink-deps %{nodejs_sitelib}
```

Comment 4 Gwyn Ciesla 2020-04-17 14:01:09 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/yarnpkg

Comment 5 Zuzana Svetlikova 2020-04-19 22:47:41 UTC
I removed the %{nodejs_symlink_deps} and built the package.

https://src.fedoraproject.org/rpms/yarnpkg

https://koji.fedoraproject.org/koji/buildinfo?buildID=1494973


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