[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread HonzaCholasta
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

HonzaCholasta commented:
"""
master:

* 417f1926c48b426b34b18edb28869f4f06824873 spec file: support client-only build
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286742894
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread HonzaCholasta
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

HonzaCholasta commented:
"""
Now that PR #502 was pushed this should finally work.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286729368
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread pvomacka
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

pvomacka commented:
"""
ipa server build without tests does not work, so NACK
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286728170
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread pvomacka
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

pvomacka commented:
"""
@HonzaCholasta Thank you for update - only client build on Fedora now works. So 
ACK once the travis pass.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286720580
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread lslebodn
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

lslebodn commented:
"""
On (15/03/17 02:32), Jan Cholasta wrote:
>I can't, both are required to build client components (`ipa-getkeytab` 
>specifically), moving them to the server section would break client-only RPM 
>build.
>
Sorry for confusion. I misinterpreted changes in diff.

The real change was moving following part to server only build
```
 # %{_unitdir}, %{_tmpfilesdir}
 BuildRequires:  systemd
 # systemd-tmpfiles which is executed from make install requires apache user
 BuildRequires:  httpd
```

But git diff decided to show moving of other lines

LS

"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286694086
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread pvomacka
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

pvomacka commented:
"""
Client only build does not work on Fedora. So NACK.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286692657
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

tiran commented:
"""
```ipa-getkeytab``` uses ```libini_config``` and ```libsasl2```.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286687873
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread HonzaCholasta
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

HonzaCholasta commented:
"""
I can't, both are required to build client components (`ipa-getkeytab` 
specifically), moving them to the server section would break client-only RPM 
build.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286687860
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#596][comment] spec file: support client-only build

2017-03-15 Thread lslebodn
  URL: https://github.com/freeipa/freeipa/pull/596
Title: #596: spec file: support client-only build

lslebodn commented:
"""
You can also move few dependencies to server only build
```
BuildRequires:  libini_config-devel
BuildRequires:  cyrus-sasl-devel
```

Check spec file changes in **rejected** PR 
https://github.com/freeipa/freeipa/pull/494/files

otherwise LGTM

"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/596#issuecomment-286683090
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code