Skip to content

Maintainance cleanup #20

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jnpkrn wants to merge 16 commits into from
Closed

Maintainance cleanup #20

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
4e4a3ec
build: s/CPPFLAGS/CFLAGS/ + use GLIB_LIB
jnpkrn Feb 17, 2016
056b9c9
maint: spec: SPDX license notation only for SUSE
jnpkrn Feb 17, 2016
e40d4a3
maint: spec: drop redundant, unused %check scriptlet
jnpkrn Feb 17, 2016
ebf53e3
maint: spec: get rid of redundant %clean section
jnpkrn Feb 17, 2016
ab5f397
maint: configure: drop useless AC_CHECK_LIB invocations
jnpkrn Feb 17, 2016
ceec5c2
maint: spec: use pkgconfig for glib-2.0
jnpkrn Feb 17, 2016
e030217
maint: configure: nicely formatted help/summary screens
jnpkrn Feb 17, 2016
2c22f54
maint: configure: --enable-resource-monitor no longer relevant
jnpkrn Feb 24, 2016
1319660
maint: build: allow requesting no HTML man + use in spec
jnpkrn Feb 24, 2016
5d2b6b3
Low: avoid using cl_log directly
jnpkrn Feb 17, 2016
1dcd750
Low: "char *" more string funcs friendly than unsigned variant
jnpkrn Feb 23, 2016
b8b9657
Low: attr: assert the string truncation will not occur
jnpkrn Feb 24, 2016
ce1effc
Low: make (a defensive) use of known static buffer sizes
jnpkrn Feb 24, 2016
89d9e06
Low: apparently static function should be marked so
jnpkrn Feb 24, 2016
8149495
Low: const-cast char arrays where suitable
jnpkrn Feb 23, 2016
9c7ccd2
Low: use unsigned int where suitable
jnpkrn Feb 23, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
maint: spec: use pkgconfig for glib-2.0 
...as this is the only reference configure script uses.

References:
http://rpm.org/gitweb?p=rpm.git;a=commit;h=6c4b0fc (~2007)
  • Loading branch information
@jnpkrn
jnpkrn committed Apr 11, 2016
commit ceec5c267c46d01a5fc7b498c7637f10bc885b06
6 changes: 5 additions & 1 deletion booth.spec
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,12 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-build
BuildRequires: asciidoc
BuildRequires: autoconf
BuildRequires: automake
BuildRequires: pkgconfig
%if 0%{?suse_version}
BuildRequires: glib2-devel
%else
BuildRequires: pkgconfig(glib-2.0)
%fi
Copy link

@dmuhamedagic dmuhamedagic Apr 18, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be %endif, right?

Copy link
Author

@jnpkrn jnpkrn Apr 19, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On 18/04/16 08:21 -0700, Dejan Muhamedagic wrote:

BuildRequires: glib2-devel
+%else
+BuildRequires: pkgconfig(glib-2.0)
+%fi

This should be %endif, right?

Right, thanks for noticing.
(I blame vim highlighter for showing "%if" the same way as "%fi").

Jan (Poki)

BuildRequires: libgcrypt-devel
%if 0%{?fedora} || 0%{?centos} || 0%{?rhel}
BuildRequires: cluster-glue-libs-devel
Expand All @@ -50,7 +55,6 @@ BuildRequires: libglue-devel
BuildRequires: libpacemaker-devel
%endif
BuildRequires: libxml2-devel
BuildRequires: pkgconfig
%if 0%{?fedora} || 0%{?centos} || 0%{?rhel}
Requires: pacemaker >= 1.1.8
Requires: cluster-glue-libs >= 1.0.6
Expand Down