[PD-dev] [ pure-data-Bugs-2829640 ] Makefile use "pd_src" for path-to-pd-without-src

SourceForge.net noreply at sourceforge.net
Tue Apr 13 06:47:27 CEST 2010


Bugs item #2829640, was opened at 2009-07-30 09:15
Message generated for change (Settings changed) made by eighthave
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2829640&group_id=55736

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: pd-extended
Group: None
>Status: Closed
Resolution: None
Priority: 5
Private: No
Submitted By: IOhannes m zmlnig (zmoelnig)
Assigned to: Nobody/Anonymous (nobody)
Summary: Makefile use "pd_src" for path-to-pd-without-src

Initial Comment:
several makefiles use the "pd_src" variable to point to /where/is/my/pd/ rather than /where/is/my/pd/src.

this makes the code unreadable, as the variable name suggests that it holds the path to the Pd-sources, esp when it comes to $(pd_src)/src and similar

please use a more general variable name for this, e.g. "PD_PATH"
(i would suggest upper-case according to http://www.gnu.org/software/make/manual/make.html#toc_Using-Variables).

done for the ext13 skeleton Makefile

----------------------------------------------------------------------

>Comment By: Hans-Christoph Steiner (eighthave)
Date: 2010-04-13 00:47

Message:
PD_PATH is used in the Makefile template, so I am closing this bug.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave)
Date: 2009-09-20 12:28

Message:
I am fine with a clearer naming scheme, but it would take more than just
this little change.  'pd_src' goes with 'externals_src',
'abstractions_src', 'doc-src', etc.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478070&aid=2829640&group_id=55736




More information about the Pd-dev mailing list