ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
On (23/05/14 17:47), Lukas Slebodnik wrote:
ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
The patch lost somewhere.
One more time
LS
On (23/05/14 17:50), Lukas Slebodnik wrote:
On (23/05/14 17:47), Lukas Slebodnik wrote:
ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
The patch lost somewhere.
One more time
LS
I found out there is already similar ticket. Attached patch also get rid of Fedora 17 and older conditionals 1 file changed, 6 insertions(+), 76 deletions(-) :-)
LS
PS: With recent patches[1][2][3][4], it is possible to build sssd on rhel5. (without samba and without python bindings). The spec file would be very complicated. I have a modified spec in my private branch. If we want we can have spec file for epel5 in contrib directory.
[1] b3007e32fa5d6b722f3aaaf9fe7593103cb443c3 [2] 1ecb16023779f912ba1c4e9e418acb59296ce8bc [3] 18647db9e275e7aa3e002551237abae756a2afb9 [4] a2e417f38c57ed87c956ddcecf4dafca93842b65
On 06/06/2014 02:29 PM, Lukas Slebodnik wrote:
On (23/05/14 17:50), Lukas Slebodnik wrote:
On (23/05/14 17:47), Lukas Slebodnik wrote:
ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
The patch lost somewhere.
One more time
LS
I found out there is already similar ticket. Attached patch also get rid of Fedora 17 and older conditionals 1 file changed, 6 insertions(+), 76 deletions(-) :-)
LS
PS: With recent patches[1][2][3][4], it is possible to build sssd on rhel5. (without samba and without python bindings). The spec file would be very complicated. I have a modified spec in my private branch. If we want we can have spec file for epel5 in contrib directory.
[1] b3007e32fa5d6b722f3aaaf9fe7593103cb443c3 [2] 1ecb16023779f912ba1c4e9e418acb59296ce8bc [3] 18647db9e275e7aa3e002551237abae756a2afb9 [4] a2e417f38c57ed87c956ddcecf4dafca93842b65
What a nice spec file it is! :)
I have just one question, this diff actually seems to add a condition intead of remove it: BuildRequires: glib2-devel BuildRequires: selinux-policy-targeted -%if (0%{?fedora} >= 18) +%if 0%{?fedora} BuildRequires: libcmocka-devel +%endif +%if (0%{?fedora} || 0%{?rhel} >= 7) BuildRequires: libnl3-devel %else BuildRequires: libnl-devel @@ -139,10 +115,7 @@ BuildRequires: systemd-devel BuildRequires: cifs-utils-devel %endif
Can you move it to a separate commit?
On (12/06/14 14:03), Pavel Březina wrote:
On 06/06/2014 02:29 PM, Lukas Slebodnik wrote:
On (23/05/14 17:50), Lukas Slebodnik wrote:
On (23/05/14 17:47), Lukas Slebodnik wrote:
ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
The patch lost somewhere.
One more time
LS
I found out there is already similar ticket. Attached patch also get rid of Fedora 17 and older conditionals 1 file changed, 6 insertions(+), 76 deletions(-) :-)
LS
PS: With recent patches[1][2][3][4], it is possible to build sssd on rhel5. (without samba and without python bindings). The spec file would be very complicated. I have a modified spec in my private branch. If we want we can have spec file for epel5 in contrib directory.
[1] b3007e32fa5d6b722f3aaaf9fe7593103cb443c3 [2] 1ecb16023779f912ba1c4e9e418acb59296ce8bc [3] 18647db9e275e7aa3e002551237abae756a2afb9 [4] a2e417f38c57ed87c956ddcecf4dafca93842b65
What a nice spec file it is! :)
I have just one question, this diff actually seems to add a condition intead of remove it: BuildRequires: glib2-devel BuildRequires: selinux-policy-targeted -%if (0%{?fedora} >= 18) +%if 0%{?fedora} BuildRequires: libcmocka-devel +%endif +%if (0%{?fedora} || 0%{?rhel} >= 7) BuildRequires: libnl3-devel %else BuildRequires: libnl-devel @@ -139,10 +115,7 @@ BuildRequires: systemd-devel BuildRequires: cifs-utils-devel %endif
Can you move it to a separate commit?
Sure, new version is attached.
LS
On 06/12/2014 03:49 PM, Lukas Slebodnik wrote:
On (12/06/14 14:03), Pavel Březina wrote:
On 06/06/2014 02:29 PM, Lukas Slebodnik wrote:
On (23/05/14 17:50), Lukas Slebodnik wrote:
On (23/05/14 17:47), Lukas Slebodnik wrote:
ehlo,
In theory, it could be possible to build current master without samba on rhel5, but the spec file would be very complicated. It is better to simplify spec file.
Patch is attached. 1 file changed, 43 deletions(-) :-)
LS
The patch lost somewhere.
One more time
LS
I found out there is already similar ticket. Attached patch also get rid of Fedora 17 and older conditionals 1 file changed, 6 insertions(+), 76 deletions(-) :-)
LS
PS: With recent patches[1][2][3][4], it is possible to build sssd on rhel5. (without samba and without python bindings). The spec file would be very complicated. I have a modified spec in my private branch. If we want we can have spec file for epel5 in contrib directory.
[1] b3007e32fa5d6b722f3aaaf9fe7593103cb443c3 [2] 1ecb16023779f912ba1c4e9e418acb59296ce8bc [3] 18647db9e275e7aa3e002551237abae756a2afb9 [4] a2e417f38c57ed87c956ddcecf4dafca93842b65
What a nice spec file it is! :)
I have just one question, this diff actually seems to add a condition intead of remove it: BuildRequires: glib2-devel BuildRequires: selinux-policy-targeted -%if (0%{?fedora} >= 18) +%if 0%{?fedora} BuildRequires: libcmocka-devel +%endif +%if (0%{?fedora} || 0%{?rhel} >= 7) BuildRequires: libnl3-devel %else BuildRequires: libnl-devel @@ -139,10 +115,7 @@ BuildRequires: systemd-devel BuildRequires: cifs-utils-devel %endif
Can you move it to a separate commit?
Sure, new version is attached.
LS
Hi, I'm sorry for the delay, the patch does no longer apply. Can you rebase it please?
On Thu, Sep 04, 2014 at 01:05:14PM +0200, Pavel Březina wrote:
On 09/03/2014 04:52 PM, Lukas Slebodnik wrote:
On (04/08/14 11:52), Pavel Březina wrote:
Hi, I'm sorry for the delay, the patch does no longer apply. Can you rebase it please?
Rebased patch is attached.
LS
Ack.
master: * 2344d7f71dd80618a41745b0818b46895fa61b2c * 3e7aeece252e088b2be14c9545a211d6543fd66f
sssd-devel@lists.fedorahosted.org