There are changes with samba-3.0.23 that don't appear to addresses by the current ebuild: xml, ldapsam, and sql backends are gone from samba proper with sql backends available separately ebuild diff attached - this will not handle sql backends but does fix bug #134477 Chris
Created attachment 91591 [details, diff] diff to handle some samba 3.0.23 changes
You lost $(use_with kerberos ads) in your patch.
How about reading the changelog?
(In reply to comment #2) > You lost $(use_with kerberos ads) in your patch. > I don't think it's needed. I you set the kerberos use flag samba will automatically configure for ads.
(In reply to comment #3) > How about reading the changelog? > Why doesn't the ebuild reflect the changes? Seems that it should. Or am I mistaken? The unused use flags are still there. Configure options that no longer exist are being passed. And there is no download of the third party source for sql backends (and again the use flags are still there for them).
well, in your patch you lost a bit of ldap... but you're right: those use flags have to be removed. I just wanted to wait for xml and *sql support requests: for now, the ebuild doesn't support them. In a few days, if nobody raises the issue, they will be removed. If the use flags are present, however, there will be no problem during compilation/install
(In reply to comment #6) > well, in your patch you lost a bit of ldap... I'm missing why you say this as ldapsam to support samba-2.2 ldap compatibility is now kaput. Plus, the way it was previously handled caused bug #134477 as --without-ldap would never get sent to configure. I suppose the ads use flag could have been left in but if kerberos and ldap are set ads support will build automatically. I suppose someone may want kerberos and ldap support without ads but it's possibly unlikely, although it may be best to retain such flexibility. Even though I removed the 'mymods' section, $(use_with ldap) was added. Did I miss something vital regarding ldap? Of course, I sidestepped the whole sql backend issue by removing those sections but that issue is beyond my skill level :) And BTW, I appreciate all of the excellent work you do maintaining the package, especially getting these releases out on a timely basis. Not trying to get in your hair, just trying to assist when I can. Chris
I'd like to keep the "ads" check. On all the other parts, I like your suggestions: implementing them in 3.0.23a (no one complained about the *sql and xml backends missing). Reporting here a small talk already had privately: an ebuild is GPL-ed, so it belongs to all the community. Everyone has the right to enhance it: packages require sometimes a lot of steps to install, and if a user has a good idea on one of these steps, it is very welcomed.
