[suggest] icinga 1.4.0

[Michael Friedrich]

Hi,

just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).

Thanks,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: icinga.spec
Url: http://lists.repoforge.org/pipermail/users/attachments/20110511/f8fd4f2b/attachment.pl

[David Hrbáč]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the
spec file shipping within the tarball to match 1.4.0 as version and
latest changes. it runs smoothly and creates all required pkg.
I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).
Thanks,
Michael

Michael,
I reviewed your spec file. Looks good to me, so I have committed it.
Thanks,
DH

[Michael Friedrich]

Hi,

Post by David Hrbáč
Michael,
I reviewed your spec file. Looks good to me, so I have committed it.

Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Attached git diff undos that.

Sorry for the inconvenience causing that, I was not aware of that change :(

Thanks,
Michael

PS: If you prefer GIT patches in the future, just let me know. I've just
cloned the git repo and put the patch against master.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-update-icinga.spec-re-use-icingacmd-group-instead-fo.patch
Type: text/x-diff
Size: 2331 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110511/01524a19/attachment.bin

[David Hrbáč]

Post by Michael Friedrich
Hi,
Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Committed. Your patch is missing:
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Post by Michael Friedrich
PS: If you prefer GIT patches in the future, just let me know. I've
just cloned the git repo and put the patch against master.

I don't mind. Feel free to use git or patch. :o)
DH

[Michael Friedrich]

Post by David Hrbáč
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Ouch. Thanks for clearing this up!

Post by David Hrbáč
I don't mind. Feel free to use git or patch. :o)

Ok thanks a lot for the fast actions, much appreciated! =)

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[David Hrbáč]

Post by Michael Friedrich
Ok thanks a lot for the fast actions, much appreciated! =)

Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)
DH

[Michael Friedrich]

Post by David Hrbáč
Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)

That's just fine. I want to make sure that during release preparations
various packaging systems get their updated stuff (and i don't forget
things).

Those users not being able to wait still can use the provided
icinga.spec file within the tarball and create their own packages ;)
I've applied your patch against icinga git master and r1.4 already, so
it will be in the release tarball.

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)

That's just fine. I want to make sure that during release preparations
various packaging systems get their updated stuff (and i don't forget
things).

Those users not being able to wait still can use the provided
icinga.spec file within the tarball and create their own packages ;)
I've applied your patch against icinga git master and r1.4 already, so
it will be in the release tarball.

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)

That's just fine. I want to make sure that during release preparations
various packaging systems get their updated stuff (and i don't forget
things).

Those users not being able to wait still can use the provided
icinga.spec file within the tarball and create their own packages ;)
I've applied your patch against icinga git master and r1.4 already, so
it will be in the release tarball.

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)

That's just fine. I want to make sure that during release preparations
various packaging systems get their updated stuff (and i don't forget
things).

Those users not being able to wait still can use the provided
icinga.spec file within the tarball and create their own packages ;)
I've applied your patch against icinga git master and r1.4 already, so
it will be in the release tarball.

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)

That's just fine. I want to make sure that during release preparations
various packaging systems get their updated stuff (and i don't forget
things).

Those users not being able to wait still can use the provided
icinga.spec file within the tarball and create their own packages ;)
I've applied your patch against icinga git master and r1.4 already, so
it will be in the release tarball.

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[David Hrbáč]

Post by Michael Friedrich
Ok thanks a lot for the fast actions, much appreciated! =)

Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Ok thanks a lot for the fast actions, much appreciated! =)

Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Ok thanks a lot for the fast actions, much appreciated! =)

Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Ok thanks a lot for the fast actions, much appreciated! =)

Michael,
Commit might be fast, but when the package is going to be released no
one knows. Dag might know. :o)
DH

[Michael Friedrich]

Post by David Hrbáč
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Ouch. Thanks for clearing this up!

Post by David Hrbáč
I don't mind. Feel free to use git or patch. :o)

Ok thanks a lot for the fast actions, much appreciated! =)

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Ouch. Thanks for clearing this up!

Post by David Hrbáč
I don't mind. Feel free to use git or patch. :o)

Ok thanks a lot for the fast actions, much appreciated! =)

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Ouch. Thanks for clearing this up!

Post by David Hrbáč
I don't mind. Feel free to use git or patch. :o)

Ok thanks a lot for the fast actions, much appreciated! =)

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[Michael Friedrich]

Post by David Hrbáč
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Ouch. Thanks for clearing this up!

Post by David Hrbáč
I don't mind. Feel free to use git or patch. :o)

Ok thanks a lot for the fast actions, much appreciated! =)

Kind regards,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

[David Hrbáč]

Post by Michael Friedrich
Hi,
Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Committed. Your patch is missing:
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Post by Michael Friedrich
PS: If you prefer GIT patches in the future, just let me know. I've
just cloned the git repo and put the patch against master.

I don't mind. Feel free to use git or patch. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Hi,
Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Committed. Your patch is missing:
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Post by Michael Friedrich
PS: If you prefer GIT patches in the future, just let me know. I've
just cloned the git repo and put the patch against master.

I don't mind. Feel free to use git or patch. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Hi,
Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Committed. Your patch is missing:
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Post by Michael Friedrich
PS: If you prefer GIT patches in the future, just let me know. I've
just cloned the git repo and put the patch against master.

I don't mind. Feel free to use git or patch. :o)
DH

[David Hrbáč]

Post by Michael Friedrich
Hi,
Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Committed. Your patch is missing:
@@ -12,7 +12,7 @@
Summary: Open Source host, service and network monitoring program
Name: icinga
Version: 1.4.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv2+
Group: Applications/System
URL: http://www.icinga.org/

Post by Michael Friedrich
PS: If you prefer GIT patches in the future, just let me know. I've
just cloned the git repo and put the patch against master.

I don't mind. Feel free to use git or patch. :o)
DH

[Michael Friedrich]

Hi,

Post by David Hrbáč
Michael,
I reviewed your spec file. Looks good to me, so I have committed it.

Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Attached git diff undos that.

Sorry for the inconvenience causing that, I was not aware of that change :(

Thanks,
Michael

PS: If you prefer GIT patches in the future, just let me know. I've just
cloned the git repo and put the patch against master.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-update-icinga.spec-re-use-icingacmd-group-instead-fo.patch
Type: text/x-diff
Size: 2331 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110511/01524a19/attachment-0001.bin

[Michael Friedrich]

Hi,

Post by David Hrbáč
Michael,
I reviewed your spec file. Looks good to me, so I have committed it.

Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Attached git diff undos that.

Sorry for the inconvenience causing that, I was not aware of that change :(

Thanks,
Michael

PS: If you prefer GIT patches in the future, just let me know. I've just
cloned the git repo and put the patch against master.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-update-icinga.spec-re-use-icingacmd-group-instead-fo.patch
Type: text/x-diff
Size: 2331 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110511/01524a19/attachment-0002.bin

[Michael Friedrich]

Hi,

Post by David Hrbáč
Michael,
I reviewed your spec file. Looks good to me, so I have committed it.

Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Attached git diff undos that.

Sorry for the inconvenience causing that, I was not aware of that change :(

Thanks,
Michael

PS: If you prefer GIT patches in the future, just let me know. I've just
cloned the git repo and put the patch against master.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-update-icinga.spec-re-use-icingacmd-group-instead-fo.patch
Type: text/x-diff
Size: 2331 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110511/01524a19/attachment-0003.bin

[Michael Friedrich]

Hi,

Post by David Hrbáč
Michael,
I reviewed your spec file. Looks good to me, so I have committed it.

Just one thing we just discussed about, reviewing the final diff you
just committed. It came up that previous commits were changing the
command group and would break upgrade procedure.

Attached git diff undos that.

Sorry for the inconvenience causing that, I was not aware of that change :(

Thanks,
Michael

PS: If you prefer GIT patches in the future, just let me know. I've just
cloned the git repo and put the patch against master.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-update-icinga.spec-re-use-icingacmd-group-instead-fo.patch
Type: text/x-diff
Size: 2331 bytes
Desc: not available
URL: <http://lists.repoforge.org/pipermail/users/attachments/20110511/01524a19/attachment-0004.bin>

[Jose Pedro Oliveira]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

Michael,

What's the reason for versioning the virtual nagios provides?

Specfile diff from 1.3.1-1 to 1.4.0-2:
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------

This seems to be rather wrong/misleading as the current upstream nagios
version is 3.2.3 and the above line creates a nagios 1.4.0 provides:

----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------

Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

Regards,
jpo

[Michael Friedrich]

Post by David Hrbáč
Michael,
What's the reason for versioning the virtual nagios provides?

Hmmm, no particular reasons. This was added a while ago in GIT tree and
I probably did not see it amongst the various diffs I already reverted.

Post by David Hrbáč
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------
This seems to be rather wrong/misleading as the current upstream nagios
----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------
Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

most likely you're right - thanks for the pointer!

https://dev.icinga.org/issues/1562

either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

thanks,
Michael

Post by David Hrbáč
Regards,
jpo

[Michael Friedrich]

Post by Michael Friedrich
either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

attached.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-undo-provides-nagios-version.patch
Type: text/x-diff
Size: 1296 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110518/69546d17/attachment.bin

[Michael Friedrich]

Post by Michael Friedrich
either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

attached.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-undo-provides-nagios-version.patch
Type: text/x-diff
Size: 1296 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110518/69546d17/attachment-0001.bin

[Michael Friedrich]

Post by Michael Friedrich
either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

attached.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-undo-provides-nagios-version.patch
Type: text/x-diff
Size: 1296 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110518/69546d17/attachment-0002.bin

[Michael Friedrich]

Post by Michael Friedrich
either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

attached.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-undo-provides-nagios-version.patch
Type: text/x-diff
Size: 1296 bytes
Desc: not available
Url : http://lists.repoforge.org/pipermail/users/attachments/20110518/69546d17/attachment-0003.bin

[Michael Friedrich]

Post by Michael Friedrich
either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

attached.
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-undo-provides-nagios-version.patch
Type: text/x-diff
Size: 1296 bytes
Desc: not available
URL: <http://lists.repoforge.org/pipermail/users/attachments/20110518/69546d17/attachment-0004.bin>

[Michael Friedrich]

Post by David Hrbáč
Michael,
What's the reason for versioning the virtual nagios provides?

Hmmm, no particular reasons. This was added a while ago in GIT tree and
I probably did not see it amongst the various diffs I already reverted.

Post by David Hrbáč
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------
This seems to be rather wrong/misleading as the current upstream nagios
----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------
Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

most likely you're right - thanks for the pointer!

https://dev.icinga.org/issues/1562

either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

thanks,
Michael

Post by David Hrbáč
Regards,
jpo

[Michael Friedrich]

Post by David Hrbáč
Michael,
What's the reason for versioning the virtual nagios provides?

Hmmm, no particular reasons. This was added a while ago in GIT tree and
I probably did not see it amongst the various diffs I already reverted.

Post by David Hrbáč
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------
This seems to be rather wrong/misleading as the current upstream nagios
----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------
Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

most likely you're right - thanks for the pointer!

https://dev.icinga.org/issues/1562

either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

thanks,
Michael

Post by David Hrbáč
Regards,
jpo

[Michael Friedrich]

Post by David Hrbáč
Michael,
What's the reason for versioning the virtual nagios provides?

Hmmm, no particular reasons. This was added a while ago in GIT tree and
I probably did not see it amongst the various diffs I already reverted.

Post by David Hrbáč
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------
This seems to be rather wrong/misleading as the current upstream nagios
----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------
Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

most likely you're right - thanks for the pointer!

https://dev.icinga.org/issues/1562

either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

thanks,
Michael

Post by David Hrbáč
Regards,
jpo

[Michael Friedrich]

Post by David Hrbáč
Michael,
What's the reason for versioning the virtual nagios provides?

Hmmm, no particular reasons. This was added a while ago in GIT tree and
I probably did not see it amongst the various diffs I already reverted.

Post by David Hrbáč
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------
This seems to be rather wrong/misleading as the current upstream nagios
----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------
Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

most likely you're right - thanks for the pointer!

https://dev.icinga.org/issues/1562

either someone could do it by hand for rpmforge, or i'll provide the
diff as soon as i get my hands back on the git.

thanks,
Michael

Post by David Hrbáč
Regards,
jpo

[Michael Friedrich]

Hi,

just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).

Thanks,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: icinga.spec
Url: http://lists.repoforge.org/pipermail/users/attachments/20110511/f8fd4f2b/attachment-0001.pl

[David Hrbáč]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the
spec file shipping within the tarball to match 1.4.0 as version and
latest changes. it runs smoothly and creates all required pkg.
I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).
Thanks,
Michael

Michael,
I reviewed your spec file. Looks good to me, so I have committed it.
Thanks,
DH

[Jose Pedro Oliveira]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

Michael,

What's the reason for versioning the virtual nagios provides?

Specfile diff from 1.3.1-1 to 1.4.0-2:
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------

This seems to be rather wrong/misleading as the current upstream nagios
version is 3.2.3 and the above line creates a nagios 1.4.0 provides:

----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------

Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

Regards,
jpo

[Michael Friedrich]

Hi,

just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).

Thanks,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: icinga.spec
Url: http://lists.repoforge.org/pipermail/users/attachments/20110511/f8fd4f2b/attachment-0002.pl

[David Hrbáč]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the
spec file shipping within the tarball to match 1.4.0 as version and
latest changes. it runs smoothly and creates all required pkg.
I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).
Thanks,
Michael

Michael,
I reviewed your spec file. Looks good to me, so I have committed it.
Thanks,
DH

[Jose Pedro Oliveira]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

Michael,

What's the reason for versioning the virtual nagios provides?

Specfile diff from 1.3.1-1 to 1.4.0-2:
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------

This seems to be rather wrong/misleading as the current upstream nagios
version is 3.2.3 and the above line creates a nagios 1.4.0 provides:

----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------

Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

Regards,
jpo

[Michael Friedrich]

Hi,

just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).

Thanks,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: icinga.spec
Url: http://lists.repoforge.org/pipermail/users/attachments/20110511/f8fd4f2b/attachment-0003.pl

[David Hrbáč]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the
spec file shipping within the tarball to match 1.4.0 as version and
latest changes. it runs smoothly and creates all required pkg.
I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).
Thanks,
Michael

Michael,
I reviewed your spec file. Looks good to me, so I have committed it.
Thanks,
DH

[Jose Pedro Oliveira]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

Michael,

What's the reason for versioning the virtual nagios provides?

Specfile diff from 1.3.1-1 to 1.4.0-2:
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------

This seems to be rather wrong/misleading as the current upstream nagios
version is 3.2.3 and the above line creates a nagios 1.4.0 provides:

----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------

Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

Regards,
jpo

[Michael Friedrich]

Hi,

just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).

Thanks,
Michael
--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: michael.friedrich at univie.ac.at
phone: [phone]
mobile: [phone]
fax: [phone]
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: icinga.spec
URL: <http://lists.repoforge.org/pipermail/users/attachments/20110511/f8fd4f2b/attachment.ksh>

[David Hrbáč]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the
spec file shipping within the tarball to match 1.4.0 as version and
latest changes. it runs smoothly and creates all required pkg.
I'm new to rpmforge package commits, so please excuse my question on
what is especially provided (Chris Maser did it before, but now he left).
What steps would be required next - the current spec file is attached,
the source tarball can be provided / will be on sourceforge at 11:00
local vienna time (~2,5h).
Thanks,
Michael

Michael,
I reviewed your spec file. Looks good to me, so I have committed it.
Thanks,
DH

[Jose Pedro Oliveira]

Post by Michael Friedrich
Hi,
just because of getting the release into rpmforge, I've updated the spec
file shipping within the tarball to match 1.4.0 as version and latest
changes. it runs smoothly and creates all required pkg.

Michael,

What's the reason for versioning the virtual nagios provides?

Specfile diff from 1.3.1-1 to 1.4.0-2:
-----------
@@ -28,7 +28,7 @@
BuildRequires: libjpeg-devel
BuildRequires: libdbi-devel
BuildRequires: perl(ExtUtils::Embed)
-Provides: nagios
+Provides: nagios = %{version}
?
%description
Icinga is an application, system and network monitoring application.
------------

This seems to be rather wrong/misleading as the current upstream nagios
version is 3.2.3 and the above line creates a nagios 1.4.0 provides:

----------
$ rpm -qp --provides icinga-1.4.0-2.el6.x86_64.rpm | grep nagios
nagios = 1.4.0
----------

Shouldn't it be better to revert this line change?
(i.e., ship an unversioned nagios provides, like in the
icinga 1.3.1 specfile)

Regards,
jpo