Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 103164 - samba 3.0.20 released, but some components are broken
Summary: samba 3.0.20 released, but some components are broken
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Gentoo's SAMBA Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-20 11:09 UTC by Chris Smith
Modified: 2005-09-01 01:14 UTC (History)
2 users (show)

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


Attachments
samba-3.0.20.ebuild (samba-3.0.20.ebuild,9.30 KB, text/plain)
2005-08-22 05:26 UTC, Christian Andreetta (RETIRED)
Details
files/003_all_samba-3.0.x-libdirsymlink.patch (003_all_samba-3.0.x-libdirsymlink.patch,622 bytes, patch)
2005-08-22 05:27 UTC, Christian Andreetta (RETIRED)
Details | Diff
files/010_all_samba-3.0.x-extrabins.patch (010_all_samba-3.0.x-extrabins.patch,1.19 KB, patch)
2005-08-22 05:27 UTC, Christian Andreetta (RETIRED)
Details | Diff
files/011_all_samba-3.0.x-subins.patch (011_all_samba-3.0.x-subins.patch,3.38 KB, patch)
2005-08-22 05:28 UTC, Christian Andreetta (RETIRED)
Details | Diff
proposed patch to above 3.0.20 ebuild (samba-3.0.20.ebuild.patch,1.28 KB, patch)
2005-08-22 10:18 UTC, Chris Smith
Details | Diff
proposed patch to above 3.0.20 ebuild (samba-3.0.20.ebuild.patch,1.02 KB, patch)
2005-08-22 11:51 UTC, Chris Smith
Details | Diff
samba-3.0.20.ebuild, patch against samba-3.0.14a-r3.ebuild (samba-3.0.20.ebuild.patch,2.94 KB, patch)
2005-08-23 05:34 UTC, Christian Andreetta (RETIRED)
Details | Diff
files/015_all_samba-3.0.20-vscan-fileaccesslog.patch (015_all_samba-3.0.20-vscan-fileaccesslog.patch,1.73 KB, patch)
2005-08-23 05:35 UTC, Christian Andreetta (RETIRED)
Details | Diff
files/016_all_samba-3.0.20-vscan-mc_daemon_core.patch (016_all_samba-3.0.20-vscan-mc_daemon_core.patch,582 bytes, patch)
2005-08-23 05:35 UTC, Christian Andreetta (RETIRED)
Details | Diff
samba-3.0.20.ebuild.patch (samba-3.0.20.ebuild.patch,347 bytes, patch)
2005-08-26 16:29 UTC, INODE64 Sistemas
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Smith 2005-08-20 11:09:30 UTC
ebuild requested for new version Samba 3.0.20

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Christian Andreetta (RETIRED) gentoo-dev 2005-08-22 05:26:01 UTC
There are some issues I'd like to clean out:

1) test functions changed in an incomplete way: the var SMB4TORTURE (proper
usage?) and the script smb4torture are missing.
2) samba-vscan module doesn't even compile

Anyway, USE="-oav" emerge -B samba will build.
I'm posting the ebuild and some of the needed patches here. If anybody wnats to
test them...
Comment 2 Christian Andreetta (RETIRED) gentoo-dev 2005-08-22 05:26:49 UTC
Created attachment 66534 [details]
samba-3.0.20.ebuild
Comment 3 Christian Andreetta (RETIRED) gentoo-dev 2005-08-22 05:27:23 UTC
Created attachment 66535 [details, diff]
files/003_all_samba-3.0.x-libdirsymlink.patch
Comment 4 Christian Andreetta (RETIRED) gentoo-dev 2005-08-22 05:27:50 UTC
Created attachment 66536 [details, diff]
files/010_all_samba-3.0.x-extrabins.patch
Comment 5 Christian Andreetta (RETIRED) gentoo-dev 2005-08-22 05:28:16 UTC
Created attachment 66537 [details, diff]
files/011_all_samba-3.0.x-subins.patch
Comment 6 Chris Smith 2005-08-22 07:53:43 UTC
Apparently there's a newer version of smbtorture in the samba 4 tree, maybe 
what smb4torture is referring to. It is a developer tool and normally built 
separately. Don't know why we would build this by default. 
 
Also wonder why we build both static and shared libraries by default? 
 
Should we have a use flag for aio support (introduced with this version)? 
Syslog support is listed as experimental yet the ebuild forces --with-syslog 
by default. Do we want to do this? 
 
Noticed that "configure --help" doesn't list "--with-idmap" yet we pass that 
as a configure option. 
Comment 7 Chris Smith 2005-08-22 10:18:12 UTC
Created attachment 66569 [details, diff]
proposed patch to above 3.0.20 ebuild

Added use flags for ads, aio and syslog.
Removed --enable-static (don't know if this is needed, but it works fine here
and the Samba default is to not build static libraries).
Comment 8 Chris Smith 2005-08-22 10:34:00 UTC
Just noticed that ads seems to be handled with the kerberos use flag. 
Maybe I'm mistaken but even though ads support requires kerberos, is the 
opposite true? IOW, can kerberos be useful to Samba if one isn't 
authenticating with an MS AD? 
Comment 9 Chris Smith 2005-08-22 11:51:50 UTC
Created attachment 66573 [details, diff]
proposed patch to above 3.0.20 ebuild

dropped the ads use flag - seems to be handled
added automount flag
Comment 10 Chris Smith 2005-08-22 12:09:14 UTC
From Samba's Whatsnew: 
 
Changes since 3.0.14a 
    * Stop using C++ reserved words so that Samba can be compiled  
      using g++.  Also allows VFS modules in C++. 
 
Maybe these changes are affecting the vscan module? 
 
Comment 11 Chris Smith 2005-08-22 12:21:16 UTC
Someone has already filed a bug on the samba-vscan sourceforge site.  
  
For some reason the ebuild wants to manually build rpctorture, which doesn't  
build. The 3.0.14a ebuild is not successful in this regard either. Maybe it  
should be dropped? 
 
At any rate, if someone doesn't need the vscan module, -oav works here. 
  
 
Comment 12 Christian Andreetta (RETIRED) gentoo-dev 2005-08-23 05:31:33 UTC
The bug in [1] was opened by me. It's now closed: the proper g++ patches were
already present in sf.net cvs.
This could also be a moment to further clean the ebuild. I like your suggestions:

-- $(use_with aio) should read $(use_with aio aio-support)
-- --with-idmap is historical or undocumented?: testing with this option removed
-- while kerberos protocol could be useful also beside ads, AFAIK samba
implementation binds the two together
-- syslog should require a logger. Included virtual/logger dependency

Still don't like, anyway, the broken test module (the smb4torture script is a
requirement for 'make test')

[1]
http://sourceforge.net/tracker/index.php?func=detail&aid=1266158&group_id=10590&atid=110590
Comment 13 Christian Andreetta (RETIRED) gentoo-dev 2005-08-23 05:34:27 UTC
Created attachment 66640 [details, diff]
samba-3.0.20.ebuild, patch against samba-3.0.14a-r3.ebuild
Comment 14 Christian Andreetta (RETIRED) gentoo-dev 2005-08-23 05:35:01 UTC
Created attachment 66641 [details, diff]
files/015_all_samba-3.0.20-vscan-fileaccesslog.patch
Comment 15 Christian Andreetta (RETIRED) gentoo-dev 2005-08-23 05:35:35 UTC
Created attachment 66642 [details, diff]
files/016_all_samba-3.0.20-vscan-mc_daemon_core.patch
Comment 16 Chris Smith 2005-08-23 07:14:35 UTC
>-- $(use_with aio) should read $(use_with aio aio-support) 
>-- --with-idmap is historical or undocumented?: 
 
Good catch. 
 
I think idmap is historical, possibly since 3.0.0 beta2. There's a note in the 
whatsnew: 
------------------------ 
Beginning with Samba3.0.0beta3, winbindd has been .... 
... 
2) All idmap lookups have been moved into winbindd. 
------------------------ 
 
"configure --help" for 3.0.14a does not mention --with-idmap either 
 
Comment 17 INODE64 Sistemas 2005-08-26 16:29:12 UTC
Created attachment 66945 [details, diff]
samba-3.0.20.ebuild.patch

update to configure with option syslog
Comment 18 Christian Andreetta (RETIRED) gentoo-dev 2005-09-01 01:14:16 UTC
in cvs. fixed bug #101675 also (non-linux compatibility)