[Bugs] [Bug 1290534] New: Minor improvements and cleanup for the build system
bugzilla at redhat.com
bugzilla at redhat.com
Thu Dec 10 18:01:10 UTC 2015
https://bugzilla.redhat.com/show_bug.cgi?id=1290534
Bug ID: 1290534
Summary: Minor improvements and cleanup for the build system
Product: GlusterFS
Version: 3.7.6
Component: build
Keywords: Tracking, Triaged
Severity: low
Priority: low
Assignee: bugs at gluster.org
Reporter: madam at redhat.com
CC: bugs at gluster.org, gluster-bugs at redhat.com,
kkeithle at redhat.com, ndevos at redhat.com
Depends On: 1198849
Blocks: 1225842
+++ This bug was initially created as a clone of Bug #1198849 +++
Use this bug to file changes that affect the build tools and configuration.
Real bugs (problem reports) should have their own bugzilla entry for related
patches.
Acceptable changes include:
- improve readability
- minor checkpatch.pl changes
- cleanup in Makefile.am files or configure.ac
- ...
--- Additional comment from Anand Avati on 2015-03-04 17:53:39 EST ---
REVIEW: http://review.gluster.org/9784 (build: adjust checkpatch for HEAD)
posted (#2) for review on master by Niels de Vos (ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-03-04 18:13:04 EST ---
REVIEW: http://review.gluster.org/9803 (build: add more generated files to
.gitignore) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-03-04 23:17:47 EST ---
REVIEW: http://review.gluster.org/9784 (build: adjust checkpatch for HEAD.)
posted (#3) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-05 16:10:48 EST ---
COMMIT: http://review.gluster.org/9784 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit 0bef7717d3100734c6c5a4ba85de7a39e76774db
Author: Michael Scherer <misc at zarb.org>
Date: Mon Mar 2 16:24:05 2015 +0100
build: adjust checkpatch for HEAD.
README is called README.md in git HEAD, and we have a directory
heal.
Change-Id: I2e698b95c6924b5b21aa54257fefd6a6698255ee
BUG: 1198849
Signed-off-by: Michael Scherer <misc at zarb.org>
Reviewed-on: http://review.gluster.org/9784
Reviewed-by: Humble Devassy Chirammal <humble.devassy at gmail.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Niels de Vos <ndevos at redhat.com>
--- Additional comment from Anand Avati on 2015-03-10 11:36:38 EDT ---
COMMIT: http://review.gluster.org/9803 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit 4f2b49eefd1a56469770534d7941fa37ae759f11
Author: Niels de Vos <ndevos at redhat.com>
Date: Wed Mar 4 16:22:39 2015 +0100
build: add more generated files to .gitignore
Added *_unittest, *.log, *.trs, extras/run-gluster.tmpfiles and
heal/src/glfsheal. Sorted the list of files to make it easier to modify
in future.
Change-Id: I0f21a03a89e61f84c6a85c8f1f98712c5819d633
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/9803
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
--- Additional comment from Anand Avati on 2015-03-18 06:02:44 EDT ---
REVIEW: http://review.gluster.org/9928 (marker: fix compile time warning on buf
arg.) posted (#1) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-18 10:44:00 EDT ---
COMMIT: http://review.gluster.org/9928 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit dbd62a8d2b50392fbed0a0781a4f241dadb8f506
Author: Humble Devassy Chirammal <hchiramm at redhat.com>
Date: Wed Mar 18 15:25:08 2015 +0530
marker: fix compile time warning on buf arg.
Problem:
marker-quota.c: In function 'mq_inspect_directory_xattr_task':
marker-quota.c:3451:31: warning: variable 'buf' set but not
used [-Wunused-but-set-variable]
struct iatt buf = {0,};
Change-Id: I211378328bdb2509a5d2a186d173f7f30a670c8a
BUG: 1198849
Signed-off-by: Humble Devassy Chirammal <hchiramm at redhat.com>
Reviewed-on: http://review.gluster.org/9928
Reviewed-by: Vijaikumar Mallikarjuna <vmallika at redhat.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
Reviewed-by: Niels de Vos <ndevos at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
--- Additional comment from Anand Avati on 2015-03-19 05:41:33 EDT ---
REVIEW: http://review.gluster.org/9940 (build: let checkpatch check for a
trailing dot) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-03-21 06:29:37 EDT ---
REVIEW: http://review.gluster.org/9958 (glupy: correct the definition of
GlusterFS in setup.py) posted (#1) for review on master by Humble Devassy
Chirammal (humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-22 08:21:39 EDT ---
REVIEW: http://review.gluster.org/9963 (build: only include sys/sysctl.h when
sysctl() is used) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-03-23 15:28:38 EDT ---
COMMIT: http://review.gluster.org/9963 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com)
------
commit 889044594614756243259a22ed1448408a2b33ac
Author: Niels de Vos <ndevos at redhat.com>
Date: Sun Mar 22 13:17:46 2015 +0100
build: only include sys/sysctl.h when sysctl() is used
BUG: 1198849
Change-Id: I9597b4b7f37994865f88b99651ea9ec89787f5cf
Reported-by: Adam Borowski <kilobyte at angband.pl>
URL: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=778790
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/9963
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
--- Additional comment from Anand Avati on 2015-03-23 16:15:41 EDT ---
COMMIT: http://review.gluster.org/9940 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com)
------
commit 2308c07b9d52b28f371388b9393b17508b2ddccd
Author: Niels de Vos <ndevos at redhat.com>
Date: Thu Mar 19 10:38:23 2015 +0100
build: let checkpatch check for a trailing dot
The subject of a patch should not end with a dot (.). It is not common
to have subjects of emails end with a dot, and neither is it common for
patches.
Change-Id: Id090241393aee3ca99df4887bdb2d7a7a8913164
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/9940
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Ira Cooper <ira at redhat.com>
Reviewed-by: Lalatendu Mohanty <lmohanty at redhat.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
--- Additional comment from Anand Avati on 2015-03-24 07:20:36 EDT ---
REVIEW: http://review.gluster.org/9958 (glupy: correct the definition of
GlusterFS in setup.py) posted (#2) for review on master by Humble Devassy
Chirammal (humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-25 10:23:20 EDT ---
REVIEW: http://review.gluster.org/9997 (features/ctr: Removing rpc-lib, rpc-xdr
and api from CPPFLAGS) posted (#1) for review on master by Anoop C S
(achiraya at redhat.com)
--- Additional comment from Anand Avati on 2015-03-25 15:02:31 EDT ---
COMMIT: http://review.gluster.org/9997 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit f64666f113e7f0d3f7061dfed7e6e24928e85161
Author: Anoop C S <achiraya at redhat.com>
Date: Wed Mar 25 19:51:41 2015 +0530
features/ctr: Removing rpc-lib, rpc-xdr and api from CPPFLAGS
Changetimerecorder doesn't seem to use rpc-lib, rpc-xdr
and gfapi in the source. So removing those from Makefile.am
Change-Id: I21c71db6212c10ba3821c6c456958a45c5312d41
BUG: 1198849
Signed-off-by: Anoop C S <achiraya at redhat.com>
Reviewed-on: http://review.gluster.org/9997
Reviewed-by: Joseph Fernandes <josferna at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Niels de Vos <ndevos at redhat.com>
--- Additional comment from Anand Avati on 2015-03-30 03:06:45 EDT ---
REVIEW: http://review.gluster.org/10040 (build: ignore cscope files) posted
(#1) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-30 06:04:32 EDT ---
REVIEW: http://review.gluster.org/10040 (build: ignore cscope files) posted
(#2) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-30 07:09:30 EDT ---
REVIEW: http://review.gluster.org/10040 (build: ignore cscope files) posted
(#3) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-03-30 08:38:13 EDT ---
COMMIT: http://review.gluster.org/9958 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com)
------
commit d2629aef8302c6b7d832063351c0ce1f2f2efbdd
Author: Humble Devassy Chirammal <hchiramm at redhat.com>
Date: Sat Mar 21 15:57:13 2015 +0530
glupy: correct the definition of GlusterFS in setup.py
Change-Id: I31597a623b4ebf3d3129067eb20c661c910b97fe
BUG: 1198849
Signed-off-by: Humble Devassy Chirammal <hchiramm at redhat.com>
Reviewed-on: http://review.gluster.org/9958
Reviewed-by: Lalatendu Mohanty <lmohanty at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
--- Additional comment from Anand Avati on 2015-03-30 12:44:20 EDT ---
COMMIT: http://review.gluster.org/10040 committed in master by Kaleb KEITHLEY
(kkeithle at redhat.com)
------
commit 0a3420dbd57361174eb30fc054754f8c5499ecfb
Author: Humble Devassy Chirammal <hchiramm at redhat.com>
Date: Mon Mar 30 12:32:27 2015 +0530
build: ignore cscope files
It is common for developers to use cscope and generate
these cscope.* files with: cscope -b -q or with some
other switches of cscope. This patch avoid 'git' to worry
about the same.
Change-Id: I3525e6f1a36f9800f4c27a2cdc6ea9c668981717
BUG: 1198849
Signed-off-by: Humble Devassy Chirammal <hchiramm at redhat.com>
Reviewed-on: http://review.gluster.org/10040
Reviewed-by: Niels de Vos <ndevos at redhat.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
--- Additional comment from Anand Avati on 2015-04-01 07:51:24 EDT ---
REVIEW: http://review.gluster.org/10099 (build: sort gitignore entries) posted
(#1) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-04-01 13:53:05 EDT ---
REVIEW: http://review.gluster.org/10099 (build: sort gitignore entries) posted
(#2) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-04-02 00:00:11 EDT ---
REVIEW: http://review.gluster.org/10099 (build: sort gitignore entries) posted
(#3) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-04-06 05:37:41 EDT ---
REVIEW: http://review.gluster.org/10132 (build: add more files to .gitignore)
posted (#1) for review on master by Niels de Vos (ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-04-06 09:00:43 EDT ---
COMMIT: http://review.gluster.org/10132 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit dc128c6bb0f8935e0aaa64602cd8a2dcedaebbf4
Author: Niels de Vos <ndevos at redhat.com>
Date: Mon Apr 6 15:01:15 2015 +0530
build: add more files to .gitignore
Change-Id: Icef0d7f443f7caf3aa386d3a6978f98cf3a5a4af
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/10132
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
--- Additional comment from Anand Avati on 2015-04-06 09:02:18 EDT ---
COMMIT: http://review.gluster.org/10099 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit 120cdbf3ba51f493df3bff112a9e21c19349b543
Author: Humble Devassy Chirammal <hchiramm at redhat.com>
Date: Mon Mar 30 12:49:13 2015 +0530
build: sort gitignore entries
Change-Id: I79d315efb47577ab88d090a96df13f1f92ed276c
BUG: 1198849
Signed-off-by: Humble Devassy Chirammal <hchiramm at redhat.com>
Reviewed-on: http://review.gluster.org/10099
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Niels de Vos <ndevos at redhat.com>
--- Additional comment from Anand Avati on 2015-05-21 12:01:53 EDT ---
REVIEW: http://review.gluster.org/10885 (spec: correct the vendor string in
spec file) posted (#1) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-05-22 03:37:43 EDT ---
REVIEW: http://review.gluster.org/10885 (spec: correct the vendor string in
spec file) posted (#2) for review on master by Humble Devassy Chirammal
(humble.devassy at gmail.com)
--- Additional comment from Anand Avati on 2015-05-23 13:45:51 EDT ---
REVIEW: http://review.gluster.org/10897 (build: do not include rpcgen generated
files in "make dist") posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-05-23 13:52:15 EDT ---
REVIEW: http://review.gluster.org/10897 (build: do not include rpcgen generated
files in "make dist") posted (#2) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-05-23 14:07:10 EDT ---
REVIEW: http://review.gluster.org/10897 (build: do not include rpcgen generated
files in "make dist") posted (#3) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-05-28 05:35:57 EDT ---
COMMIT: http://review.gluster.org/10885 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit bc33fe3c1621612788cf2f7928178f314a5f25f1
Author: Humble Devassy Chirammal <hchiramm at redhat.com>
Date: Thu May 21 21:30:54 2015 +0530
spec: correct the vendor string in spec file
Change-Id: Id19c973092af506582741887d12fe00c85022f15
BUG: 1198849
Signed-off-by: Humble Devassy Chirammal <hchiramm at redhat.com>
Reviewed-on: http://review.gluster.org/10885
Tested-by: NetBSD Build System
Reviewed-by: Niels de Vos <ndevos at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
--- Additional comment from Anand Avati on 2015-05-31 17:18:52 EDT ---
REVIEW: http://review.gluster.org/11020 (rpm: add option "--with debug" for
rpmbuild/mock) posted (#1) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-06-01 05:18:41 EDT ---
REVIEW: http://review.gluster.org/11020 (rpm: add option "--with debug" for
rpmbuild/mock) posted (#2) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-06-04 19:20:55 EDT ---
REVIEW: http://review.gluster.org/11020 (rpm: add option "--with debug" for
rpmbuild/mock) posted (#3) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-06-06 12:51:15 EDT ---
REVIEW: http://review.gluster.org/11020 (rpm: add option "--with debug" for
rpmbuild/mock) posted (#4) for review on master by Vijay Bellur
(vbellur at redhat.com)
--- Additional comment from Anand Avati on 2015-06-07 14:33:49 EDT ---
COMMIT: http://review.gluster.org/11020 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit ffe8b4d8149b50df53353bdb85fac1251a85ec2f
Author: Niels de Vos <ndevos at redhat.com>
Date: Sun May 31 21:50:21 2015 +0200
rpm: add option "--with debug" for rpmbuild/mock
When passing "--with debug" to mock or rpmbuild, the build process will
add the "--enable-debug" option to ./configure. This makes it easier
to build RPMs with debugging enabled.
Change-Id: Ie584a4a03525385b337bbf81e5093f2af1b0b780
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/11020
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
--- Additional comment from Anand Avati on 2015-06-17 17:56:41 EDT ---
REVIEW: http://review.gluster.org/11297 (build: always include the VERSION file
in 'make dist') posted (#2) for review on master by Niels de Vos
(ndevos at redhat.com)
--- Additional comment from Anand Avati on 2015-06-28 10:46:58 EDT ---
COMMIT: http://review.gluster.org/11297 committed in master by Niels de Vos
(ndevos at redhat.com)
------
commit 5f76ff1beaaf1fd1c85c43a166c289f9094a379b
Author: Niels de Vos <ndevos at redhat.com>
Date: Wed Jun 17 23:33:25 2015 +0200
build: always include the VERSION file in 'make dist'
Building a 'make dist' tarball from a non-git directory fails to include
the VERSION file. This subsequently fails to set the version in the
installation directories (for the xlators) and the RPM packaging can not
find the correct files.
This change makes sure to include a VERSION file in the 'make dist'
tarball, no matter from where 'make dist' is run. It mainly addresses
this use case:
$ git clone ... && cd glusterfs
$ ./autogen.sh
$ mkdir /tmp/workdir
$ cd /tmp/workdir
$ $OLDPWD/configure
$ make dist
Change-Id: I3e87b5e16118b68d065fe99790d4b17e0cef9189
BUG: 1198849
Signed-off-by: Niels de Vos <ndevos at redhat.com>
Reviewed-on: http://review.gluster.org/11297
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Tested-by: NetBSD Build System <jenkins at build.gluster.org>
--- Additional comment from Vijay Bellur on 2015-10-13 07:58:30 EDT ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#1) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-10-15 01:13:31 EDT ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#2) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-10-15 08:35:07 EDT ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#3) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-10-17 12:30:09 EDT ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#4) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-10-28 00:54:48 EDT ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#5) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-11-03 05:12:10 EST ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#6) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-11-03 06:51:40 EST ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#7) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-11-03 08:01:45 EST ---
REVIEW: http://review.gluster.org/12351 (build: Fix autoconf warnings) posted
(#8) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-11-23 02:16:05 EST ---
REVIEW: http://review.gluster.org/12714 (build: Remove redundant options from
GF_CFLAGS) posted (#1) for review on master by Anoop C S (anoopcs at redhat.com)
--- Additional comment from Vijay Bellur on 2015-11-30 07:43:39 EST ---
REVIEW: http://review.gluster.org/12828 (extras: fix deprecation warining in
checkpatch.pl) posted (#1) for review on master by Michael Adam
(obnox at samba.org)
--- Additional comment from Vijay Bellur on 2015-12-01 12:25:49 EST ---
COMMIT: http://review.gluster.org/12714 committed in master by Jeff Darcy
(jdarcy at redhat.com)
------
commit 86701a0ececa5fdbbe08654516618baadc4555b7
Author: Anoop C S <anoopcs at redhat.com>
Date: Mon Nov 23 12:20:09 2015 +0530
build: Remove redundant options from GF_CFLAGS
While configuring, GF_CFLAGS is being populated with redudndant
options as follows:
Normal mode:
GF_CFLAGS = -g -O2 -g -O2 -Wformat -Werror=format-security \
-g -O2 -g -O2 -Wformat -Werror=format-security \
-Werror=implicit-function-declaration
Debug mode:
GF_CFLAGS = -g -O2 -g -O0 -DDEBUG -Wformat -Werror=format-security \
-g -O2 -g -O0 -DDEBUG -Wformat -Werror=format-security \
-Werror=implicit-function-declaration
This patch is to remove this redundancy of options.
Change-Id: Idf6f2536c2a8ae24ba257681fbe9dfdb8bfa4f19
BUG: 1198849
Signed-off-by: Anoop C S <anoopcs at redhat.com>
Reviewed-on: http://review.gluster.org/12714
Tested-by: NetBSD Build System <jenkins at build.gluster.org>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Reviewed-by: Kaleb KEITHLEY <kkeithle at redhat.com>
Reviewed-by: Jeff Darcy <jdarcy at redhat.com>
--- Additional comment from Vijay Bellur on 2015-12-02 07:43:36 EST ---
COMMIT: http://review.gluster.org/12828 committed in master by Jeff Darcy
(jdarcy at redhat.com)
------
commit 2e9376a2f1b992d3649dd1d5a8e844b4c031ef3c
Author: Michael Adam <obnox at samba.org>
Date: Mon Nov 30 13:35:23 2015 +0100
extras: fix deprecation warining in checkpatch.pl
Left braces in patterns need to be escaped.
Otherwise, for perl >= 5.16, we get a warning:
"Unescaped left brace in regex is deprecated, passed through in regex;"
This patch fixes the relevant braces.
Hence perl -c ./examples/checkpatch.pl is clean again.
Change-Id: I938c9c262239f53de57d30a2cff7030f22e63dc1
BUG: 1198849
Signed-off-by: Michael Adam <obnox at samba.org>
Reviewed-on: http://review.gluster.org/12828
Reviewed-by: Anoop C S <anoopcs at redhat.com>
Reviewed-by: Niels de Vos <ndevos at redhat.com>
Tested-by: NetBSD Build System <jenkins at build.gluster.org>
Tested-by: Gluster Build System <jenkins at build.gluster.com>
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1198849
[Bug 1198849] Minor improvements and cleanup for the build system
https://bugzilla.redhat.com/show_bug.cgi?id=1225842
[Bug 1225842] Minor improvements and cleanup for the build system
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
More information about the Bugs
mailing list