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

build: support lib.cc #1035

Open
wants to merge 27 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
ba22565
initial library building support
Nov 14, 2024
aef5ee2
remove debug print
Nov 14, 2024
8026e26
fmt
Nov 14, 2024
d7963cf
Merge pull request #1 from poac-dev/main
Lazauya Nov 14, 2024
739474c
Merge remote-tracking branch 'refs/remotes/origin/main' into lib-buil…
Nov 14, 2024
8765dcc
merge master
Nov 14, 2024
3cb0dc7
rollback formatting
Nov 14, 2024
bac7f94
rollback something
Nov 14, 2024
47f8ec8
use format for lib build command
Nov 15, 2024
9cc8c6e
Merge branch 'poac-dev:main' into main
Lazauya Nov 15, 2024
d75857a
Merge remote-tracking branch 'origin/main' into lib-building
Nov 15, 2024
f12e007
formatting, fix uninitialized vars
Nov 15, 2024
3967984
formatting, fix uninitialized vars
Nov 15, 2024
7fc1152
fix lint errors
Nov 15, 2024
a32a4a6
change to fmt::format
Nov 15, 2024
b822f85
format
Nov 15, 2024
b00868f
Merge branch 'poac-dev:main' into main
Lazauya Nov 16, 2024
344be4c
Merge remote-tracking branch 'origin/main' into lib-building
Nov 16, 2024
911d6e0
format
Nov 16, 2024
b28f9a9
fix build
Nov 16, 2024
9e58bf4
refactor build command into function
Nov 16, 2024
0ed1788
preallocate commands vec and make const
Nov 16, 2024
a2bebc9
change var names, introduce library name variables
Nov 21, 2024
e2da2be
Merge branch 'poac-dev:main' into main
Lazauya Nov 23, 2024
48c07e4
Merge remote-tracking branch 'refs/remotes/origin/main' into lib-buil…
Nov 23, 2024
1bc082f
move libname to BuildConfig and fix formatting
Nov 23, 2024
1452d25
combine defineLib/LinkTarget into single func, rename executable to b…
Nov 30, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
89 changes: 77 additions & 12 deletions src/BuildConfig.cc
Original file line number Diff line number Diff line change
Expand Up @@ -494,6 +494,15 @@ BuildConfig::defineLinkTarget(
defineTarget(binTarget, commands, deps);
}

void
BuildConfig::defineLibTarget(
const std::string& libTarget, const std::unordered_set<std::string>& deps
) {
std::vector<std::string> commands;
commands.emplace_back("ar rcs lib" + getPackageName() + ".a $^");
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
defineTarget(libTarget, commands, deps);
}

// Map a path to header file to the corresponding object file.
//
// e.g., src/path/to/header.h -> poac.d/path/to/header.o
Expand Down Expand Up @@ -791,6 +800,9 @@ BuildConfig::configureBuild() {
const auto isMainSource = [](const fs::path& file) {
return file.filename().stem() == "main";
};
const auto isLibSource = [](const fs::path& file) {
return file.filename().stem() == "lib";
};
fs::path mainSource;
for (const auto& entry : fs::directory_iterator(srcDir)) {
const fs::path& path = entry.path();
Expand All @@ -802,13 +814,34 @@ BuildConfig::configureBuild() {
}
if (mainSource.empty()) {
mainSource = path;
executable = true;
} else {
throw PoacError("multiple main sources were found");
}
}

if (mainSource.empty()) {
throw PoacError(fmt::format("src/main{} was not found", SOURCE_FILE_EXTS));
fs::path libSource;
for (const auto& entry : fs::directory_iterator(srcDir)) {
const fs::path& path = entry.path();
if (!SOURCE_FILE_EXTS.contains(path.extension())) {
continue;
}
if (!isLibSource(path)) {
continue;
}
if (libSource.empty()) {
libSource = path;
library = true;
} else {
throw PoacError("multiple lib sources were found");
}
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
}

if (mainSource.empty() && libSource.empty()) {
throw PoacError(fmt::format(
"neither src/main{} nor src/lib{} was not found", SOURCE_FILE_EXTS,
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
SOURCE_FILE_EXTS
));
}

if (!fs::exists(outBasePath)) {
Expand All @@ -817,8 +850,16 @@ BuildConfig::configureBuild() {

setVariables();

std::unordered_set<std::string> buildAll = {};
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
if (executable) {
buildAll.insert(packageName);
}
if (library) {
buildAll.insert("lib" + packageName + ".a");
}

// Build rules
setAll({ packageName });
setAll(buildAll);
addPhony("all");

std::vector<fs::path> sourceFilePaths = listSourceFilePaths(srcDir);
Expand All @@ -832,7 +873,16 @@ BuildConfig::configureBuild() {
"Move it directly to 'src/' if intended as such.",
sourceFilePath.string()
));
} else if (sourceFilePath != libSource && isLibSource(sourceFilePath)) {
logger::warn(fmt::format(
"source file `{}` is named `lib` but is not located directly in the "
"`src/` directory. "
"This file will not be treated as a library. "
"Move it directly to 'src/' if intended as such.",
sourceFilePath.string()
));
}

srcs += ' ' + sourceFilePath.string();
}

Expand All @@ -842,16 +892,31 @@ BuildConfig::configureBuild() {
const std::unordered_set<std::string> buildObjTargets =
processSources(sourceFilePaths);

// Project binary target.
const std::string mainObjTarget = buildOutPath / "main.o";
std::unordered_set<std::string> projTargetDeps = { mainObjTarget };
collectBinDepObjs(
projTargetDeps, "",
targets.at(mainObjTarget).remDeps, // we don't need sourceFile
buildObjTargets
);
if (executable) {
// Project binary target.
const std::string mainObjTarget = buildOutPath / "main.o";
std::unordered_set<std::string> projTargetDeps = { mainObjTarget };

defineLinkTarget(outBasePath / packageName, projTargetDeps);
collectBinDepObjs(
projTargetDeps, "",
targets.at(mainObjTarget).remDeps, // we don't need sourceFile
buildObjTargets
);

Lazauya marked this conversation as resolved.
Show resolved Hide resolved
defineLinkTarget(outBasePath / packageName, projTargetDeps);
}

if (library) {
// Project library target.
const std::string libTarget = buildOutPath / "lib.o";
std::unordered_set<std::string> libTargetDeps = { libTarget };
collectBinDepObjs(
libTargetDeps, "",
targets.at(libTarget).remDeps, // we don't need sourceFile
buildObjTargets
);
defineLibTarget(outBasePath / ("lib" + packageName + ".a"), libTargetDeps);
}

// Test Pass
std::unordered_set<std::string> testTargets;
Expand Down
17 changes: 17 additions & 0 deletions src/BuildConfig.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,11 @@ struct BuildConfig {
fs::path unittestOutPath;
bool isDebug;

// if we are building an executable
bool executable;
// if we are building a library
bool library;

std::unordered_map<std::string, Variable> variables;
std::unordered_map<std::string, std::vector<std::string>> varDeps;
std::unordered_map<std::string, Target> targets;
Expand All @@ -67,6 +72,13 @@ struct BuildConfig {
public:
explicit BuildConfig(const std::string& packageName, bool isDebug = true);

bool isExecutable() const {
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
return executable;
}
bool isLibrary() const {
Lazauya marked this conversation as resolved.
Show resolved Hide resolved
return library;
}
ken-matsui marked this conversation as resolved.
Show resolved Hide resolved

void defineVar(
const std::string& name, const Variable& value,
const std::unordered_set<std::string>& dependsOn = {}
Expand All @@ -77,12 +89,14 @@ struct BuildConfig {
varDeps[dep].push_back(name);
}
}

void defineSimpleVar(
const std::string& name, const std::string& value,
const std::unordered_set<std::string>& dependsOn = {}
) {
defineVar(name, { .value = value, .type = VarType::Simple }, dependsOn);
}

void defineCondVar(
const std::string& name, const std::string& value,
const std::unordered_set<std::string>& dependsOn = {}
Expand Down Expand Up @@ -145,6 +159,9 @@ struct BuildConfig {
void defineLinkTarget(
const std::string& binTarget, const std::unordered_set<std::string>& deps
);
void defineLibTarget(
const std::string& binTarget, const std::unordered_set<std::string>& deps
);

void collectBinDepObjs( // NOLINT(misc-no-recursion)
std::unordered_set<std::string>& deps, std::string_view sourceFileName,
Expand Down
57 changes: 41 additions & 16 deletions src/Cmd/Build.cc
Original file line number Diff line number Diff line change
Expand Up @@ -40,23 +40,48 @@ buildImpl(std::string& outDir, const bool isDebug) {
outDir = config.outBasePath;

const std::string& packageName = getPackageName();
const Command makeCmd = getMakeCommand().addArg("-C").addArg(outDir).addArg(
(config.outBasePath / packageName).string()
);
Command checkUpToDateCmd = makeCmd;
checkUpToDateCmd.addArg("--question");

int exitCode = execCmd(checkUpToDateCmd);
if (exitCode != EXIT_SUCCESS) {
// If packageName binary is not up-to-date, compile it.
logger::info(
"Compiling",
fmt::format(
"{} v{} ({})", packageName, getPackageVersion().toString(),
getProjectBasePath().string()
)
int exitCode = 0;
if (config.isExecutable()) {
const Command makeCmd = getMakeCommand().addArg("-C").addArg(outDir).addArg(
(config.outBasePath / packageName).string()
);
Command checkUpToDateCmd = makeCmd;
checkUpToDateCmd.addArg("--question");

exitCode = execCmd(checkUpToDateCmd);
if (exitCode != EXIT_SUCCESS) {
// If packageName binary is not up-to-date, compile it.
logger::info(
"Compiling",
fmt::format(
"{} v{} ({})", packageName, getPackageVersion().toString(),
getProjectBasePath().string()
)
);
exitCode = execCmd(makeCmd);
}
}

if (config.isLibrary()) {
std::string libName = "lib" + packageName + ".a";
const Command makeCmd = getMakeCommand().addArg("-C").addArg(outDir).addArg(
(config.outBasePath / libName).string()
);
exitCode = execCmd(makeCmd);
Command checkUpToDateCmd = makeCmd;
checkUpToDateCmd.addArg("--question");

exitCode = execCmd(checkUpToDateCmd);
if (exitCode != EXIT_SUCCESS) {
// If packageName binary is not up-to-date, compile it.
logger::info(
"Compiling",
fmt::format(
"{} v{} ({})", libName, getPackageVersion().toString(),
getProjectBasePath().string()
)
);
exitCode = execCmd(makeCmd);
}
}

const auto end = std::chrono::steady_clock::now();
Expand Down
Loading