Skip to content

Commit 1928752

Browse files
lint: fix black error report
1 parent 7d33545 commit 1928752

File tree

5 files changed

+20
-10
lines changed

5 files changed

+20
-10
lines changed

src/outpost/barbican/_internals/cargo_config.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ def run_cargo_config(rustargs: Path, target: Path, extra_args: str, outdir: Path
3535

3636
(outdir / ".cargo" / "config.toml").write_text(config)
3737

38+
3839
def argument_parser() -> ArgumentParser:
3940
parser = ArgumentParser()
4041
parser.add_argument("--rustargs-file", type=Path, help="rustargs file path")

src/outpost/barbican/_internals/gen_task_metadata_bin.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,4 +105,6 @@ def argument_parser() -> ArgumentParser:
105105
def run(argv: T.List[str]) -> None:
106106
args = argument_parser().parse_args(argv)
107107
# XXX: use builddir as option
108-
run_gen_task_metadata_bin(args.input, args.output, ProjectPath.load(args.projectdir / "build"))
108+
run_gen_task_metadata_bin(
109+
args.input, args.output, ProjectPath.load(args.projectdir / "output" / "build")
110+
)

src/outpost/barbican/barbican.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -156,15 +156,14 @@ def setup(self) -> None:
156156
package.installed_targets[0],
157157
package.dummy_linked_targets[0],
158158
dummy_linker_script,
159-
package_name=package.name if package.backend == Backend.Meson else "kernel"
159+
package_name=package.name if package.backend == Backend.Meson else "kernel",
160160
)
161161

162162
layout_sys_exelist = []
163163
layout_app_exelist = []
164164
for package in self._packages:
165165
if package.is_sys_package:
166166
layout_sys_exelist.extend(package.installed_targets)
167-
#elif package.backend == Backend.Meson:
168167
else:
169168
layout_app_exelist.extend(package.dummy_linked_targets)
170169

@@ -183,7 +182,6 @@ def setup(self) -> None:
183182

184183
# gen_ld/relink/gen_meta/objcopy app(s)
185184
for package in self._packages:
186-
#if package.is_application and package.backend == Backend.Meson:
187185
if package.is_application:
188186
# XXX: Handle multiple exe package
189187
elf_in = package.installed_targets[0]

src/outpost/barbican/buildsys/ninja_backend.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -282,8 +282,8 @@ def add_cargo_rules(self, rustargs: Path, rust_target: Path) -> None:
282282
description="cargo compile $name",
283283
pool="console",
284284
command="cd $builddir "
285-
+ " && config=$config $cargo build --manifest-path=$sourcedir/Cargo.toml --release"
286-
+ " && touch $out && cd -",
285+
+ " && config=$config $cargo build --manifest-path=$sourcedir/Cargo.toml --release"
286+
+ " && touch $out && cd -",
287287
)
288288
self._ninja.newline()
289289
self._ninja.rule(
@@ -302,14 +302,18 @@ def add_cargo_package(self, package: "Package") -> None:
302302
"cmd": "cargo_config",
303303
"args": f"--rustargs-file={str(package._parent._kernel.rustargs)} "
304304
+ f"--target-file={str(package._parent._kernel.rust_target)} "
305-
+ f"--extra-args=\"" + " ".join(package.build_options) + "\" "
305+
+ f'--extra-args="'
306+
+ " ".join(package.build_options)
307+
+ '" '
306308
+ f"{str(package.build_dir)}",
307309
"description": f"cargo config {package.name}",
308310
},
309311
order_only=[f"{dep}_install.stamp" for dep in package.deps],
310312
)
311313
self._ninja.newline()
312-
self._ninja.build(f"{package.name}_setup", "phony", f"{package.build_dir}/.cargo/config.toml")
314+
self._ninja.build(
315+
f"{package.name}_setup", "phony", f"{package.build_dir}/.cargo/config.toml"
316+
)
313317
self._ninja.newline()
314318
self._ninja.build(
315319
f"{package.name}_compile.stamp",
@@ -339,7 +343,7 @@ def add_cargo_package(self, package: "Package") -> None:
339343
"args": f"--suffix=.elf "
340344
+ f"--target-file={str(package._parent._kernel.rust_target)} "
341345
+ f"{str(package.build_dir)} "
342-
+ " ".join((str(t.with_suffix('')) for t in package.installed_targets)),
346+
+ " ".join((str(t.with_suffix("")) for t in package.installed_targets)),
343347
"description": f"cargo install {package.name}",
344348
},
345349
)

src/outpost/barbican/package/cargo.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,12 @@ def publish(self, *, name: str, version: str, manifest: Path, target_dir: Path)
7171
crate_index_filepath = self.index / name[:2] / name[2:4] / name
7272
if crate_index_filepath.exists():
7373
crate_index_filepath.unlink()
74-
self._cargo.package(manifest_path=str(manifest), target_dir=str(target_dir), no_verify=True, allow_dirty=True)
74+
self._cargo.package(
75+
manifest_path=str(manifest),
76+
target_dir=str(target_dir),
77+
no_verify=True,
78+
allow_dirty=True,
79+
)
7580
self._cargo.index(
7681
subcmd=["add"],
7782
crate=str(target_dir / "package" / crate_filename),

0 commit comments

Comments
 (0)