This incredibly simple patch leads to XEvil compiling and installing on AMD64 platforms, which the custom configuration system used to not do (as "x86_64" as a system identifier was not in its database). I will be testing to see if this leads to any bizarre behavior on the game's part. Reproducible: Always Steps to Reproduce: 1. epatch my patch into the ebuild 2. add "~amd64" to keywords 3. emerge xevil Actual Results: - Expected Results: -
Created attachment 178010 [details, diff] Adds knowledge of AMD64 systems to XEvil's build system Let me know if any issues arise because of this patch.
Don't mark something resolved that isn't checked into the tree, please.
The patch is clearly bogus. It's not build-system's job to add flags like -O2 -mtune=native -pipe.
The patch shouldn't be adding CFLAGS. Let me see if I can fix that.
Created attachment 254173 [details, diff] Add x86_64 tag to configure.mk Hello Frank, I tried to compile xevil and it fails, so I search here and found it. I finally got it compiled adding a x86_64 tag on configure.mk telling it have to do the same as on i386.
Excuse me about my english. As I tried to say in the last comment, the patch I've submitted add a rule for x86_64 that call i386 rule, and doing it, xevil compiles and works fine on x86_64 machines (I'm playing it right now). Please, test it if you can.
Hello? Anybody there?
(In reply to comment #7) > Hello? Anybody there? > Yes but we have other bugs to deal with as well.
(In reply to comment #8) > (In reply to comment #7) > > Hello? Anybody there? > > > > Yes but we have other bugs to deal with as well. > I know you have lot of bugs and I'm really pleased with all of you for mantain this great distribution, but this could be solved and you'll have one less :P Thanks you very much for your attention and your work.
Hi all, happy new year. Does anybody test it?
Comment on attachment 254173 [details, diff] Add x86_64 tag to configure.mk there shouldnt need to be any arch tags in any build files
Hello SpanKY, happy new year. Then, --- if use amd64; then make i386 fi --- should be added in the ebuild, but I don't know why you don't want to add it since there are lot of tags in the makefile. These are the two ways to solve it, and it should be solve as this bugs is opened 2 years ago.
no ... there should not be any arch-specific targets at all in the ebuild or in the build system. if there is, it needs to be fixed at the build system level, not by hacking things up further. your change is a workaround/hack, not a fix. i'm interested in a fix. look at how Debian builds. it simply sets FORCEHOSTTYPE first.
Re-add amd64 when there is a correct patch.
It seems all distributions already killed this, even Debian (that is ebuild is based on) removed it due to incompatibilities with gcc6
It seems someone applied some patches to make Xevil compile on 64-bit systems: https://github.com/lvella/xevil (which is linked from https://www.xevil.com/download.html ) Notice: I haven't tried to compile it myself.
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=fc9559077a12195996a8fa8879a15618a492f804 commit fc9559077a12195996a8fa8879a15618a492f804 Author: Michał Górny <mgorny@gentoo.org> AuthorDate: 2018-07-28 12:14:38 +0000 Commit: Michał Górny <mgorny@gentoo.org> CommitDate: 2018-07-28 12:15:42 +0000 games-arcade/xevil: Remove last-rited pkg Closes: https://bugs.gentoo.org/254510 games-arcade/xevil/Manifest | 2 - .../xevil/files/xevil-2.02_p2-glibc-2.10.patch | 20 ---------- games-arcade/xevil/metadata.xml | 8 ---- games-arcade/xevil/xevil-2.02_p2-r1.ebuild | 43 ---------------------- profiles/package.mask | 5 --- 5 files changed, 78 deletions(-)