| Summary: | app-editors/gvim - dropped mips keywords | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Jim Ramsay (lack) (RETIRED) <lack> |
| Component: | Current packages | Assignee: | MIPS Porters <mips> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | maxposedon, vim |
| Priority: | High | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
|
Description
Jim Ramsay (lack) (RETIRED)
2010-07-20 14:19:12 UTC
vim-7.2.442 cross compiles for mips after adding... # vim vim_cv_toupper_broken=no vim_cv_terminfo=yes vim_cv_tty_group=world vim_cv_getcwd_broken=no vim_cv_stat_ignores_slash=yes vim_cv_memmove_handles_overlap=yes to mips-linux-gnu in crossdev site will tomorrow test how it runs on Ben Nanonote, but should be fine. confirm that 7.2.442 successfully runs on mips (Ben Nanonote), plz add ~mips keyword === I have no name!@BenNanoNote / # uname -a Linux BenNanoNote 2.6.32.10 #1 PREEMPT Tue Jun 15 17:53:33 CEST 2010 mips GNU/Linux I have no name!@BenNanoNote / # vim --version VIM - Vi IMproved 7.2 (2008 Aug 9, compiled Aug 18 2010 01:40:47) Included patches: 1-442 Modified by Gentoo-7.2.442 Compiled by root@max_posedon (In reply to comment #2) > confirm that 7.2.442 successfully runs on mips (Ben Nanonote), plz add ~mips > keyword So, to be perfectly clear: Are ebuild changes necessary to add those 'vim_cv'... things you mention in comment #1? Or does adding ~mips to the current ebuild fulfill all requirements? As a further request, if you wouldn't mind also checking the new vim-7.3 and gvim-7.3, that would be nice. The new 7.3 version is basically 7.2.442 with some more bugfixes and a couple new features, but nothing groundbreaking or overly disruptive, so I'm pretty sure just adding ~mips will be okay - Just as sanity check would be nice. (In reply to comment #3) > So, to be perfectly clear: Are ebuild changes necessary to add those > 'vim_cv'... things you mention in comment #1? no, this vim_cv shouldn't goes to ebuild, they should goes to crossdev, and connected only with cross compilation. > Or does adding ~mips to the current ebuild fulfill all requirements? yes, just add ~mips keyword plz, nothing more > As a further request, if you wouldn't mind also checking the new vim-7.3 and > gvim-7.3, that would be nice. The new 7.3 version is basically 7.2.442 with > some more bugfixes and a couple new features, but nothing groundbreaking or > overly disruptive, so I'm pretty sure just adding ~mips will be okay - Just as > sanity check would be nice. I'll happy do it when #bug 333247, will be closed. And even before, I'll try suggested patch and report results. With fix in #bug 333247 , I confirm that vim works on mips (Ben Nanonote). See comments there. any news? vim-7.3 works flawlessly on mipsel (o32 ABI) whereas vim-7.0 dies with a SIGABRT. I've just taken the liberty of booting out keywords from vim-7.0 for that reason. I am just updating the O2's portage tree, and will give it a try on big-endian MIPS before I add ~mips to the keywords. (I'm confident it'll work but best to make sure.) gvim will have to wait until I get X installed on these systems... or until I ugrade vim on the Yeeloong, which is my only MIPS machine with working X at the moment. Keywords added on vim-7.3.
I notice one of the tests failed on the O2, I'll investigate this further, but manual testing of vim showed that it did work enough to be useful. It passed all tests on the other systems however... and once I rebuild the O2 I'll retry vim there too. Sadly portage does not save ${T}/build.log when src_test fails, but I suspect the problem is with the O2's somewhat dated userland.
I've been running vim-7.3 and 7.3.50 on a mips64/n32 system for as long as they've been in the tree. Looks like vim is good to go. Adjusting the summary accordingly. No idea when I'll be able to test gvim. Oh, go ahead and boot out the old versions. :) (In reply to comment #9) > Oh, go ahead and boot out the old versions. :) Done. Thanks! Oops, sorry, shouldn't have closed this, since it's still valid for gvim. Reopening. Added ~mips. |