Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 927178 - net-ftp/filezilla-3.67.0 fails to compile: filetransfer.cpp:10:6: error: type filetransferStates violates the C++ One Definition Rule [-Werror=odr]
Summary: net-ftp/filezilla-3.67.0 fails to compile: filetransfer.cpp:10:6: error: type...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks: lto
  Show dependency tree
 
Reported: 2024-03-17 08:09 UTC by Agostino Sarubbo
Modified: 2024-07-10 07:56 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
build.log (build.log,125.04 KB, text/plain)
2024-03-17 08:09 UTC, Agostino Sarubbo
Details
net-ftp/filezilla: backport patch to fix ODR violations (0001-net-ftp-filezilla-backport-patch-to-fix-ODR-violatio.patch,19.27 KB, patch)
2024-07-10 03:47 UTC, Eli Schwartz
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2024-03-17 08:09:49 UTC
https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/

Issue: net-ftp/filezilla-3.66.5 fails to compile.
Discovered on: amd64 (internal ref: lto_tinderbox)
System: LTO-SYSTEM (https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#LTO)

Info about the issue:
https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#CF0014
Comment 1 Agostino Sarubbo gentoo-dev 2024-03-17 08:09:50 UTC
Created attachment 887800 [details]
build.log

build log and emerge --info
Comment 2 Agostino Sarubbo gentoo-dev 2024-05-28 06:45:34 UTC
lto_tinderbox has reproduced this issue with version 3.67.0 - Updating summary.
Comment 3 Eli Schwartz gentoo-dev 2024-07-08 00:19:15 UTC
While I can reproduce the ODR violation, I cannot report a bug on upstream's trac installation. Account registration is broken:

Configuration Error

Cannot find implementation(s) of the IAccountRegistrationInspector interface named RegistrationFilterAdapter. Please check that the Component is enabled or update the option [account-manager] register_check in trac.ini.


...

I have sent an email to the contact address, we shall see how that goes.
Comment 4 Eli Schwartz gentoo-dev 2024-07-10 03:19:32 UTC
Their trac configuration has been fixed, and also a fix for the LTO has been committed:

https://svn.filezilla-project.org/filezilla?view=revision&revision=11167
Comment 5 Yixun Lan archtester gentoo-dev 2024-07-10 03:28:49 UTC
(In reply to Eli Schwartz from comment #4)
> Their trac configuration has been fixed, and also a fix for the LTO has been
> committed:
> 
> https://svn.filezilla-project.org/filezilla?view=revision&revision=11167

I'm not sure which approach should we take? wait for new release, or just pull in the fix
Comment 6 Eli Schwartz gentoo-dev 2024-07-10 03:47:15 UTC
Created attachment 897385 [details, diff]
net-ftp/filezilla: backport patch to fix ODR violations

The backport was trivial as it happens. I was thinking to commit this. What do you think?
Comment 7 Yixun Lan archtester gentoo-dev 2024-07-10 04:49:28 UTC
(In reply to Eli Schwartz from comment #6)
> Created attachment 897385 [details, diff] [details, diff]
> net-ftp/filezilla: backport patch to fix ODR violations
> 
> The backport was trivial as it happens. I was thinking to commit this. What
> do you think?

indeed it's quite trivial, should be safe to commit, feel free to push it, thanks
Comment 8 Larry the Git Cow gentoo-dev 2024-07-10 07:56:20 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6c33a435474c05703d8f93eed28d639b8269616e

commit 6c33a435474c05703d8f93eed28d639b8269616e
Author:     Eli Schwartz <eschwartz@gentoo.org>
AuthorDate: 2024-07-10 03:15:12 +0000
Commit:     Yixun Lan <dlan@gentoo.org>
CommitDate: 2024-07-10 07:55:49 +0000

    net-ftp/filezilla: backport patch to fix ODR violations
    
    Closes: https://bugs.gentoo.org/927178
    Signed-off-by: Eli Schwartz <eschwartz@gentoo.org>
    Signed-off-by: Yixun Lan <dlan@gentoo.org>

 net-ftp/filezilla/files/filezilla-3.67.0-odr.patch | 670 +++++++++++++++++++++
 net-ftp/filezilla/filezilla-3.67.0-r1.ebuild       |  77 +++
 2 files changed, 747 insertions(+)