[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#790816: RFS: roxterm/3.0.1-1



* Tony Houghton <h@realh.co.uk>, 2015-07-03, 16:30:
The way I get the parallel option looks quite nasty too, is there a better way to do that?

The code currently reads:

ifneq (,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
NUMJOBS = $(patsubst parallel=%,%,$(filter parallel=%,$(DEB_BUILD_OPTIONS)))
CONFIGURE_COMMON += --parallel=$(NUMJOBS)
endif

You could probably avoid repeating the filter part:

NUMJOBS = $(or $(patsubst parallel=%,%,$(filter parallel=%,$(DEB_BUILD_OPTIONS))),1)
CONFIGURE_COMMON += --parallel=$(NUMJOBS)

--
Jakub Wilk


Reply to: