From owner-freebsd-office@freebsd.org Tue Feb 26 14:10:01 2019 Return-Path: Delivered-To: freebsd-office@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 3CEBA151AD66 for ; Tue, 26 Feb 2019 14:10:01 +0000 (UTC) (envelope-from lwhsu.freebsd@gmail.com) Received: from mail-yw1-f67.google.com (mail-yw1-f67.google.com [209.85.161.67]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 55F83875E6; Tue, 26 Feb 2019 14:10:00 +0000 (UTC) (envelope-from lwhsu.freebsd@gmail.com) Received: by mail-yw1-f67.google.com with SMTP id 189so5238484ywi.3; Tue, 26 Feb 2019 06:10:00 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=nurQ9q2DRHMUq/B+ZY2TvNlYcwpvFmrPDtKBgvg2+1w=; b=gXLJ3RTBi9J2yQQ5/ehV+VmUr7psN5x+IQQSUjbO2gPfX33rZ5r3OURgvzQAD8lIp+ r34FI/bLUPcgDIOtEuVyJdECjRhZmJdoeed51mnCFG9DL7PCVwmVSnp3plx1VprMXgWS 2gz46U3SniLJ8tpwYBSTfE0dXvvGO+1Ft/w7jIEZbxkMg2REGEnRE8/mpvFIiiqMnncp gShS6ny9ZdiqN8oUjYxXuB47yaC261Pfql2fx/hXFnfBZtMuEv+3kRcM4qOxuS5UloT3 X98TJBJQpYONH3pU3XX8lYdFOj9CuCN56yttb4IYinzUAOpetgIoLotEcZvBI0Atm4wi PmPQ== X-Gm-Message-State: AHQUAuZTykDbgyOa0bTekmJJLLlLRF+DyHTgzpwMpnlRIAqiI9BIHx3U ykbvuNMB2W5M8L9GmL4oc2Mw+2Pd8ay07tT9s8A= X-Google-Smtp-Source: AHgI3IaRRKjA1URRsX/9TNftVokxnuc4JqUA84M3LvZfO+PQGS4P7+cPiTcGGF0TygCe5UIpy9+GzvgU3UVFcCjdJl0= X-Received: by 2002:a0d:c6c5:: with SMTP id i188mr17983794ywd.320.1551188319862; Tue, 26 Feb 2019 05:38:39 -0800 (PST) MIME-Version: 1.0 References: <20190218144218.GA98237@aurora.gregv.net> <20190218150917.GB98237@aurora.gregv.net> <20190219170520.GC98237@aurora.gregv.net> In-Reply-To: <20190219170520.GC98237@aurora.gregv.net> From: Li-Wen Hsu Date: Tue, 26 Feb 2019 21:38:28 +0800 Message-ID: Subject: Re: [call for help] libreoffice 6.2 update To: Greg Veldman Cc: freebsd-office@freebsd.org, Greg V , Jung-uk Kim , Justin Hibbits Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: 55F83875E6 X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; spf=pass (mx1.freebsd.org: domain of lwhsufreebsd@gmail.com designates 209.85.161.67 as permitted sender) smtp.mailfrom=lwhsufreebsd@gmail.com X-Spamd-Result: default: False [-4.08 / 15.00]; ARC_NA(0.00)[]; NEURAL_HAM_MEDIUM(-1.00)[-0.998,0]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; R_SPF_ALLOW(-0.20)[+ip4:209.85.128.0/17]; IP_SCORE(-1.18)[ipnet: 209.85.128.0/17(-3.81), asn: 15169(-2.00), country: US(-0.07)]; MIME_GOOD(-0.10)[text/plain]; MIME_TRACE(0.00)[0:+]; DMARC_NA(0.00)[freebsd.org]; RCPT_COUNT_FIVE(0.00)[5]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; TO_MATCH_ENVRCPT_SOME(0.00)[]; MX_GOOD(-0.01)[cached: alt3.gmail-smtp-in.l.google.com]; NEURAL_HAM_SHORT(-0.90)[-0.897,0]; RCVD_IN_DNSWL_NONE(0.00)[67.161.85.209.list.dnswl.org : 127.0.5.0]; RCVD_TLS_LAST(0.00)[]; FORGED_SENDER(0.30)[lwhsu@freebsd.org,lwhsufreebsd@gmail.com]; R_DKIM_NA(0.00)[]; FREEMAIL_ENVFROM(0.00)[gmail.com]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US]; TAGGED_FROM(0.00)[]; FROM_NEQ_ENVFROM(0.00)[lwhsu@freebsd.org,lwhsufreebsd@gmail.com]; RCVD_COUNT_TWO(0.00)[2] X-BeenThere: freebsd-office@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Office applications on FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Feb 2019 14:10:01 -0000 On Wed, Feb 20, 2019 at 1:05 AM Greg Veldman wrote: > > On Tue, Feb 19, 2019 at 02:50:15AM +0900, Li-Wen Hsu wrote: > > Thanks very much for the help! Dropping these patches and focusing on > > the supported FreeBSD versions sounds very reasonable to me. > > OK, I have a clean build, and I've pushed my changes up to your > GitHub repo. Highlights include: > > - bump to 6.2.1.1 > - remove patch-bridges_source_cpp__uno_gcc3__linux__intel_share.hxx > and patch-bridges_source_cpp__uno_gcc3__linux__intel_uno2cpp.cxx > as these appear to be primarily for building on now-unsupported > FreeBSD versions > - update graphics/libetonyek01 to use mdds-1.4 > - in commit c53de0f of your repo, it appears you're trying to > update the Makefile to account for the fact that > emojione-color-font is now deprecated - I updated to reflect > the fact that twemoji-color-font does not provide that file > but instead provides TwitterColorEmoji-SVGinOT.ttf > - in commit d4d3766 of upstream LibreOffice, several variables > in sal/osl/unx/thread.cxx were declared to be const. However > in function osl_thread_priority_init_Impl() they are assigned > values based on the return of various sched_get_priority_*() > functions. I'm not a C++ expert, but I believe this to be > an upstream bug. Note that on Linux all of this code is > skipped due to the definition of NO_PTHREAD_PRIORITY in > sal/osl/unx/system.hxx, so the compiler error wouldn't show > up. For now I've reverted that part of the commit. If there's > a cleaner way to do this, let me know. If you agree it's > actually a bug, I'll submit it upstream. > > I've tested that these changes build/run on my 11.2 box. I'll > try on some other systems as I have time, but I wanted to get > the changes I've been working on pushed back up to your repo > in case anyone else wants to take a look, and also to protect > against any possible loss. ;-) Thanks very much for all these work! I haven't had time to get a deeper test, but it also builds and runs fine on my 13-CURRENT system. Here are some quick feedback from me: - Now libreoffice doesn't build on i386 systems: /wrkdirs/usr/ports/editors/libreoffice/work/libreoffice-6.2.1.1/bridges/source/cpp_uno/gcc3_linux_intel/except.cxx:167:32: error: unknown type name '__si_class_type_info' rtti = new __si_class_type_info( ^ /wrkdirs/usr/ports/editors/libreoffice/work/libreoffice-6.2.1.1/bridges/source/cpp_uno/gcc3_linux_intel/except.cxx:173:32: error: unknown type name '__class_type_info' rtti = new __class_type_info( strdup( rttiName ) ); ^ 2 errors generated. It seems because we removed the two patch-bridges_source_cpp__uno_gcc3__linux__intel_* files, but currently I don't know why this affect 32-bit systems. The full error log is available at: https://gist.github.com/lwhsu/957db6755bacc2ee086b2fae513e5406 - For the assigning values to const variables issue (commit d4d3766 of LibreOffice), it also sounds a bug to me. I would suggest open an issue at upstream and see what they reply. - Haven't checked Calc can use libnumbertext correctly. I'll try to do that later this week. - There are some minor fixes pushed to the repository. BTW, if this makes your development easier, please just commit whatever you have to the repository for avoiding dropping single big commit, or feel free to new branches. Thanks for the work again! Li-Wen