search

hsutter / cppfront

A personal experimental C++ Syntax 2 -> Syntax 1 compiler
Other
5.24k stars 223 forks source link

Update source paths in README #993

Closed danra closed 4 months ago

gregmarr commented 4 months ago

Is it needed to specify the path in the commands, or should we just say that these should be built from the source directory?

danra commented 4 months ago

It's probably preferrable for the binary to be built outside of the source directory. So just specifiying the path makes sense to me. It's also less "explaining" and more "just works" than the alternative of instructing to change into the source folder.

gregmarr commented 4 months ago

On the other hand, I'd hate to imply that you HAVE to build it from the root of the install if it can be built from inside the directory.

danra commented 4 months ago

On the other hand, I'd hate to imply that you HAVE to build it from the root of the install if it can be built from inside the directory.

I don't think that's implied... these are instructions for first-time comers to the repo. Having it "just work" makes it just a bit easier, probably saving approx. a minute.

hsutter commented 4 months ago

Thanks!