Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 580732

Summary: dev-lang/moarvm: version bump to 2016.04
Product: Gentoo Linux Reporter: M. B. <tomboy64>
Component: [OLD] LibraryAssignee: Patrick Lauer <patrick>
Status: RESOLVED FIXED    
Severity: normal CC: esigra, perl
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 504706, 504710, 576284    
Attachments: moarvm.patch
moarvm.patch
nqp-2016.03-r1.patch
rakudo-2016.03-r1.patch
0001-dev-lang-moarvm-verbump-2016.04.patch
0002-dev-lang-moarvm-mask-use-flag-asan.patch
0003-dev-lang-nqp-revbump-to-2016.04.patch
0004-dev-lang-rakudo-verbump-to-2016.04.patch
complete.patch
complete.patch
complete.patch
complete.patch

Description M. B. 2016-04-21 13:49:57 UTC
Created attachment 431442 [details, diff]
moarvm.patch

The build for dev-lang/moarvm doesn't reflect the options provided by its build "system". It also links against locally compiled 3rd party libs. The whole thing doesn't seem to have seen any actual updates since 09/2014.

Note: This fixes bug 580728.
Note: This breaks compilation of rakudo.
Comment 1 Mike Gilbert gentoo-dev 2016-04-21 13:55:27 UTC
Your patch is touching a bunch of unrelated ebuilds.
Comment 2 M. B. 2016-04-21 14:22:43 UTC
Created attachment 431450 [details, diff]
moarvm.patch

Indeed. That patch I botched up.
Next try.
Comment 3 M. B. 2016-04-22 21:21:38 UTC
Created attachment 431590 [details, diff]
nqp-2016.03-r1.patch

    dev-lang/nqp: revbump to 2016.03-r1

    - Copyright adjusted
    - Tests re-enabled
    - EAPI6 enabled
    - multi-threaded compilation re-enabled
    - multi-threaded testing re-enabled

    Package-Manager: portage-2.2.28
Comment 4 M. B. 2016-04-22 21:23:13 UTC
Created attachment 431592 [details, diff]
rakudo-2016.03-r1.patch

    dev-lang/rakudo: revbump to 2016.03-r1

    make rakudo compile for MoarVM with some ugly hacks.
    and I do mean ugly.
    goddammit upstream, get your shit in order!

    Package-Manager: portage-2.2.28

Note: building with java backend fails. but rakudo with moar backend works just fine.
Comment 5 M. B. 2016-04-26 06:04:11 UTC
Created attachment 432024 [details, diff]
0001-dev-lang-moarvm-verbump-2016.04.patch
Comment 6 M. B. 2016-04-26 06:05:33 UTC
Created attachment 432026 [details, diff]
0002-dev-lang-moarvm-mask-use-flag-asan.patch
Comment 7 M. B. 2016-04-26 06:06:31 UTC
Created attachment 432028 [details, diff]
0003-dev-lang-nqp-revbump-to-2016.04.patch
Comment 8 M. B. 2016-04-26 06:08:37 UTC
Created attachment 432030 [details, diff]
0004-dev-lang-rakudo-verbump-to-2016.04.patch

These 4 files bump rakudo and its deps to 2016.04. They include numerous fixes, enhancements etc. Please test.

I'd like to commit those within a week.
Comment 9 Kent Fredric (IRC: kent\n) (RETIRED) gentoo-dev 2016-04-26 06:56:19 UTC
(In reply to M. B. from comment #4)
>     goddammit upstream, get your shit in order!

Personally, I don't find embedding these statements of emotional engagement immortally in a source repository something we should be encouraging.
Comment 10 M. B. 2016-04-26 14:54:43 UTC
Indeed it's not got any place there. I never meant it to be published, in particular since upstream is quite accomodating regarding patches.

Also, it needs further tweaking since the patches fail upstream's Travis CI.
Comment 11 M. B. 2016-04-27 00:47:33 UTC
Created attachment 432200 [details, diff]
complete.patch

These are my summarized changes.
For it to apply, you might need to remove the bottom-most patch, since it affects profiles/base/package.use.mask.

Patches have been submitted upstream and are pending review.
Comment 12 M. B. 2016-04-28 17:05:14 UTC
Created attachment 432440 [details, diff]
complete.patch

This includes updates for the 9999 ebuilds
Comment 13 Patrick Lauer gentoo-dev 2016-05-02 14:57:46 UTC
Since the java bits plain don't work I've ignored that part.

The rest, I've mostly ignored too ;)
Comment 14 M. B. 2016-05-04 18:43:47 UTC
Created attachment 433222 [details]
complete.patch

As I mentioned on IRC, I've put too much work on this patchset to have it discarded like this. Also given the amount of bugs that would be fixed with this update, this bug is far from fixed.

This update fixes the following bugs:
bug 504710
bug 504718
bug 510706
bug 576284
Comment 15 M. B. 2016-05-04 19:24:31 UTC
(In reply to Patrick Lauer from comment #13)
> Since the java bits plain don't work I've ignored that part.
> 
> The rest, I've mostly ignored too ;)

The java bits do work. I've made the combination of previously installed rakudo and java fatal. (Possibly related to bug 510706).

I can work on it further, if you provide me with a build.log of all three. Just post them here or send them to my mail.

If you need me to, I can provide patches for one patch per file/commit for better review.

$ perl6-j --version
This is Rakudo version 2016.04 built on JVM
implementing Perl 6.c.
tboy@hatshepsut /usr/portage/dev-lang/rakudo $ perl6-j -e 'say <<ok>>'
ok
Comment 16 M. B. 2016-05-08 10:48:05 UTC
The patch for MoarVM got accepted upstream -> not needed in 9999 and 2016.05+.
Comment 17 M. B. 2016-05-08 13:24:19 UTC
Created attachment 433650 [details, diff]
complete.patch

New patch to apply cleanly.
Comment 18 Patrick Lauer gentoo-dev 2016-05-09 08:03:39 UTC
should be fixed by the current 2016.04-r1 ebuild