Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makefile instructions need an update as most of the context is lost #18

Closed
Routhinator opened this issue Aug 30, 2022 · 2 comments
Closed

Comments

@Routhinator
Copy link

Hey there, just trying to follow your Makefile alteration instructions here in 2022, and the context from https://github.com/aspace-sim/aspace/wiki/Updating-Penn-Makefile seems to be lost.

There are a couple of points in the Pennmush Makefile that have a list of .c files now:

hdrs/cmds.h: src/cmds.c src/command.c src/cque.c src/help.c src/set.c src/sql.c Patchlevel
	@PERL@ utils/mkcmds.pl commands

hdrs/switches.h: src/SWITCHES Patchlevel
	@PERL@ utils/mkcmds.pl switches

src/switchinc.c: src/SWITCHES Patchlevel
	@PERL@ utils/mkcmds.pl switches

hdrs/funs.h: src/fun*.c src/bsd.c src/conf.c src/extmail.c src/help.c src/markup.c src/wiz.c src/sql.c Patchlevel src/cque.c
	@PERL@ utils/mkcmds.pl functions

As a result I cannot be certain how to follow the first instruction.

There is no list of *.o files in the Makefile anymore, so the second instruction cannot be followed.

The last instruction says to add the blob to the end of the makefile, which would put it as part of the htmldoc make target - which doesn't seem correct either.

@ray73864
Copy link
Collaborator

You are looking at the wrong Makefile :)

The instructions specifically say to alter src/Makefile.in but you are looking at the Makefile.in in the Pennmush root directory.

@Routhinator
Copy link
Author

... Oh. That makes sense. Sorry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants