Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Wrong architecture identifier for 64-bit RPM #3774

Closed
da2x opened this issue Sep 6, 2016 · 6 comments · Fixed by #4723
Closed

Wrong architecture identifier for 64-bit RPM #3774

da2x opened this issue Sep 6, 2016 · 6 comments · Fixed by #4723

Comments

@da2x
Copy link
Contributor

da2x commented Sep 6, 2016

Issue #1445 shouldn’t have been closed as there is actually an issue with Brave’s RPM packages.

The RPM package’s metadata identifies the architecture as “amd64” which flags it as “not installable” when x86_64 was expected. Which results in the following error message when trying to install via the default installation tools (dnf/yum):

Error: package brave-0.12.0-1.amd64 is not installable

The packages are built on Ubuntu, which explains where the incorrect identifier comes from. amd64 is the expected architecture name under Debian derivatives (Ubuntu, Mint, etc). However, on the RHEL side x86_64 is the expected architecture name.

Furthermore, the Group metadata property of the package should be set to Applications/Internet.

@da2x
Copy link
Contributor Author

da2x commented Oct 10, 2016

This is also the root cause of the upgrade conflicts on RPM systems.

@darkdh
Copy link
Member

darkdh commented Oct 11, 2016

@fduraibi
Copy link

I have just downloaded brave-0.12.4 and the same problem still exist!

This should be an easy fix, no?

@bbondy
Copy link
Member

bbondy commented Oct 12, 2016

PRs welcome

@da2x
Copy link
Contributor Author

da2x commented Oct 12, 2016

There is no documentation on how the RPMs and DEBs are made.

@bbondy
Copy link
Member

bbondy commented Oct 12, 2016

It uses electron-installer-debian and electron-installer-redhat inside tools/buildInstaller.js you can check out more info about those packages on github or NPM.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants