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

Fix implementation and use of the alternative package name lookup #232

Merged
merged 1 commit into from
Nov 27, 2018

Conversation

vkareh
Copy link
Member

@vkareh vkareh commented Nov 8, 2018

Any real_name which was found was not getting used before.

Also, when the lookup is unsuccessful, this means there is no
alternative package name. Thus we should use the hard-coded package
name as the real_name for our PackageKit installation request.

[bug #603757]
origin commit: https://gitlab.gnome.org/GNOME/file-roller/commit/95aafc3c

 lookup

Any real_name'' which was found was not getting used before.

Also, when the lookup is unsuccessful, this means there is no
/alternative/ package name.  Thus we should use the hard-coded package
name as the real_name for our PackageKit installation request.

[bug #603757]
origin commit: https://gitlab.gnome.org/GNOME/file-roller/commit/95aafc3c
@vkareh vkareh requested a review from a team November 8, 2018 18:42
@vkareh
Copy link
Member Author

vkareh commented Nov 8, 2018

This should supersede #227, and distros that use xz-utils (Debian et al) should technically update their local copy of the packages.match file to reflect this

@stokito
Copy link
Contributor

stokito commented Nov 10, 2018

BTW there is a lot of such things that can be backported to the File Roller and verse versa. As I understood the only real difference between FR and Engrampa is GUI so to avoid code duplication it would be great to extract shared library like libarchive. XFace also have it's own archive manage XArchiver and it even better in some aspects than FR/Engrampa.
Maybe we should discuss this approach instead of cherry picking some commits from FR?

Copy link
Member

@raveit65 raveit65 left a comment

Choose a reason for hiding this comment

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

looks good.

@raveit65 raveit65 requested a review from a team November 10, 2018 14:33
@raveit65
Copy link
Member

@sc0w
ready to go?

Copy link
Member

@sc0w sc0w left a comment

Choose a reason for hiding this comment

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

seems to be fine

@sc0w sc0w merged commit b1d6ced into master Nov 27, 2018
@sc0w sc0w deleted the fix-alternative-package-name-lookup branch November 27, 2018 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants