New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
net-im/slack-bin: change license #4533
Conversation
Pull Request assignment Areas affected: ebuilds net-im/slack-bin: @SpiderX, @gentoo/proxy-maint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those are really not things you should mix in a single commit, especially when it's involving such a major changes as retroactive license fix. Please split this into separate logical commits.
Ping. |
1. Metadata cleanup. 2. Slack 2.3.4 and 2.4.2 dropped, 2.1.2 kept as the last one with x86 support. Package-Manager: Portage-2.3.6, Repoman-2.3.1
1. Ebuild fixes backported to 2.1.2 Package-Manager: Portage-2.3.6, Repoman-2.3.1
1. License changed (bug #615466) Package-Manager: Portage-2.3.6, Repoman-2.3.1
@mgorny Sorry for a delay. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few more small changes needed.
Also, please make summaries more descriptive since that's the only thing people see in shortlogs. There's really no point in having one change noted in extended body if it would fully fit within the summary line.
fperms +x /opt/${MY_PN}/${MY_PN} | ||
dosym /opt/${MY_PN}/${MY_PN} /usr/bin/${MY_PN} | ||
} | ||
|
||
pkg_preinst() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is no longer necessary with the current gnome2-utils.eclass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed from all ebuilds.
QA_PREBUILT="opt/slack/slack | ||
opt/slack/resources/app.asar.unpacked/node_modules/* | ||
opt/slack/libnode.so | ||
opt/slack/libgcrypt.so.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason not to unbundle it? We have dev-libs/libgcrypt:11
for this library specifically.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libgcrypt.so.11 is not needed anymore.
I missed this when I was backporting fixes from newer ebuilds.
Removed it from 2.1.2
@@ -11,11 +11,11 @@ DESCRIPTION="Team collaboration tool" | |||
HOMEPAGE="http://www.slack.com/" | |||
SRC_URI="https://downloads.slack-edge.com/linux_releases/${MY_PN}-desktop-${PV}-amd64.deb" | |||
|
|||
LICENSE="no-source-code" | |||
LICENSE="all-rights-reserved" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This logically belongs before the backport ;-P.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I know... )
1. Bump ebuild to 2.6.3 (bug #619592) Package-Manager: Portage-2.3.6, Repoman-2.3.1
Package-Manager: Portage-2.3.6, Repoman-2.3.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, ok, I'll reorder them for you ;-P.
I'm going to accept it this time but please fix it ASAP. |
Package-Manager: Portage-2.3.5, Repoman-2.3.1