Created attachment 20208 [details] [review]
patch for viewres
These five apps copy, e.g., app-defaults/Viewres.ad to app-defaults/Viewres but don't ensure that the directory ./app-defaults exists. Thus a VPATH build (objdir != srcdir) will fail.
Historically, in the monolithic build Viewres.ad was installed as Viewres by a suitable imake rule. The reason for this certainly was the fact that an executable viewres and an app default file Viewres cannot coexist on a case insensitive filesystem.
For the modular build automake requires (sort of) to create a file Viewres that will then be installed. In order to cope with case insensitive filesystems that is now done in a subdirectory app-defaults. This, however, allows to distribute the app default files under their final names, no need to copy them, no need to ensure the existence of ./app-defaults.
Moreover, defining them as dist_appdefault_DATA avoids the necessity to mention them under EXTRA_DIST. All this simplifies Makefile.am considerably.
Attached are five small patches along these lines for the five apps mentioned above.
Created attachment 20209 [details] [review]
patch for xfd
Created attachment 20210 [details] [review]
patch for xgc
Created attachment 20211 [details] [review]
patch for xmh
Created attachment 20212 [details] [review]
patch for xsm
Thanks for the patches.
All of them were applied.