FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.

» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Fedora Build System

LinkBack Thread Tools
Old 07-10-2008, 02:01 AM
Todd Zullinger
Default DIST_DEFINES duplicated in Makefile.common?


It seems that DIST_DEFINES is duplicated in Makefile.common, being
included in RPM_DEFINES and then also being used in places where
RPM_DEFINES is used (directly, and as a part of RPM_WITH_DIRS).

Does the attached patch look reasonably sane?

Disclaimer: I've had a few rum-based beverages this evening and I
might be missing some important reason for the duplication.

Todd OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~
Religion. A daughter of Hope and Fear, explaining to Ignorance the
nature of the Unknowable.
-- Ambrose Bierce, The Enlarged Devil's Dictionary, 1906

Index: Makefile.common
================================================== =================
RCS file: /cvs/extras/common/Makefile.common,v
retrieving revision 1.105
diff -u -p -r1.105 Makefile.common
--- Makefile.common 3 Jul 2008 19:15:27 -0000 1.105
+++ Makefile.common 10 Jul 2008 01:47:05 -0000
@@ -94,11 +94,11 @@ ifndef NAME
$(error "You can not run this Makefile without having NAME defined")
ifndef VERSION
-VERSION := $(shell rpm $(RPM_DEFINES) $(DIST_DEFINES) -q --qf "%{VERSION}
" --specfile $(SPECFILE)| head -1)
+VERSION := $(shell rpm $(RPM_DEFINES) -q --qf "%{VERSION}
" --specfile $(SPECFILE)| head -1)
# the release of the package
ifndef RELEASE
-RELEASE := $(shell rpm $(RPM_DEFINES) $(DIST_DEFINES) -q --qf "%{RELEASE}
" --specfile $(SPECFILE)| head -1)
+RELEASE := $(shell rpm $(RPM_DEFINES) -q --qf "%{RELEASE}
" --specfile $(SPECFILE)| head -1)
# this is used in make patch, maybe make clean eventually.
# would be nicer to autodetermine from the spec file...
@@ -382,7 +382,7 @@ check: test-srpm

## use this to build an srpm locally
srpm: sources $(TARGETS)
+ $(RPM_WITH_DIRS) --nodeps -bs $(SPECFILE)

test-srpm: srpm

Fedora-buildsys-list mailing list

Thread Tools

All times are GMT. The time now is 01:27 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org