Let's add the ifndef to Makefile.global and change the
existing extension makefiles to

        PG_CONFIG := pg_config
        PGXS := $(shell $(PG_CONFIG) --pgxs)
        include $(PGXS)

Any objections?

Maybe the ":=" for pg_config is not necessary, "=" is fine for a simple string definition ?

Some documentation (not just code) update seems important to me. The patch I sent which describes the current status may be applied to existing branches, and something else can be written for the explicit PG_CONFIG setting.

Otherwise it looks okay AFAIC.


---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
      choose an index scan if your joining column's datatypes do not

Reply via email to