Skip to content
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/telegram-desktop-bin: bump to 1.3.10 and disable auto-updates #9218

Closed
wants to merge 3 commits into from

Conversation

henning-schild
Copy link
Contributor

No description provided.

@gentoo-bot
Copy link

Pull Request assignment

Areas affected: ebuilds
Packages affected: net-im/telegram-desktop-bin

net-im/telegram-desktop-bin: @henning-schild, @NP-Hardass, @gentoo/proxy-maint

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and ping us to reset the assignment.

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). labels Jul 13, 2018
@NP-Hardass
Copy link
Contributor

@henning-schild if this closes out the bug, your tag in the first commit should be "Closes" not "Bugs"

How exactly was this issue resolved? Do our users need to do anything specifically, or is it handled in the build automatically?

@henning-schild
Copy link
Contributor Author

The new ebuild makes sure the software is always started with the updater disabled, users just have to upgrade to that new version.

I think I have never seen Closes in that context, will check.

@henning-schild
Copy link
Contributor Author

@NP-Hardass switched to using "Closes"

S="${WORKDIR}/Telegram"

src_install() {
into /usr/lib/${PN}/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not that you need ending slash here

src_install() {
into /usr/lib/${PN}/
newbin "${S}/Telegram" ${PN}
into /usr/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the same

x11-libs/libX11
>=x11-libs/libxcb-1.10[xkb]
"
DEPEND=""
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one more nitpick: why do you need empty variable then?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point! was inherited from a previous ebuild, included a commit that drops that from all ebuilds

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@henning-schild do not forget to squash it as well

Signed-off-by: Henning Schild <henning@hennsch.de>
This version bump uses a new upstream feature to tell the binary that it
should not check for updates itself. The ebuild installs the binary to
/usr/lib/ and deploys a wrapper to /usr/bin/.

Closes: https://bugs.gentoo.org/618662
Signed-off-by: Henning Schild <henning@hennsch.de>
Signed-off-by: Henning Schild <henning@hennsch.de>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-07-15 20:35 UTC
Newest commit scanned: c86b7d8
Status: ✅ good

No issues found

@henning-schild henning-schild deleted the henning/staging5 branch July 16, 2018 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
5 participants