Feat/old template

[sivertjoe]

Un-reviewable PR? 😄
fix #68

setup, run, tally, and bench needed to be updated.

Very simple script aoc-migration that seems to work okay:

#!/bin/bash

TARGET=$1

mv $TARGET tmp
mkdir $TARGET
mv tmp/src/* $TARGET

for file in $TARGET/*; do
    cp tmp/Cargo.toml $file/Cargo.toml
    cp tmp/Cargo.lock $file/Cargo.lock
    mkdir -p $file/src
    mv $file/main.rs $file/src/main.rs
done

rm -rf tmp

Run with:
aoc-migration <target>

[seblyng]

Seems easier👍 will this make it impossible to run earlier years without manually converting to this new setup?

[sivertjoe]

Yes, unfortunately

…

________________________________ From: Sebastian Lyng Johansen ***@***.***> Sent: Thursday, November 28, 2024 4:09:21 PM To: seblj/cargo-aoc ***@***.***> Cc: Sivert Johansen ***@***.***>; Author ***@***.***> Subject: Re: [seblj/cargo-aoc] Feat/old template (PR #69) @seblj approved this pull request. Seems easier👍 will this make it impossible to run earlier years without manually converting to this new setup? — Reply to this email directly, view it on GitHub<#69 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI5H2YASIFVDKXXMNN6VFHL2C4W2DAVCNFSM6AAAAABSVJRAWOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDINRYGQYTGOJVHA>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[seblyng]

:( I guess it should be pretty simple to create a script to convert it?

[sivertjoe]

Sure, its possible, the only issue would be if you have any dependencies. It could be tricky to figure out which day users which dependency, but you could of course copy all dependencies (if any) to all days

…

________________________________ From: Sebastian Lyng Johansen ***@***.***> Sent: Thursday, November 28, 2024 4:35:48 PM To: seblj/cargo-aoc ***@***.***> Cc: Sivert Johansen ***@***.***>; Author ***@***.***> Subject: Re: [seblj/cargo-aoc] Feat/old template (PR #69) :( I guess it should be pretty simple to create a script to convert it? — Reply to this email directly, view it on GitHub<#69 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI5H2YHLDXY2YJM33LKQMID2C4Z5JAVCNFSM6AAAAABSVJRAWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGM4DCMZUGM>. You are receiving this because you authored the thread.Message ID: ***@***.***>

[seblyng]

Makes sense👍 fine by me, I'll just create a script to convert if I ever need it

It makes the code here a bit easier it seems like, so let's just do it

[seblyng]

Should probably be tokio::fs::copy(format!("{template_dir}/template.rs"), format!("{year}/{day}/src/main.rs")).await?;

Notice the src before main.rs. Because, now it looks like this:

Where the main.rs contains this:

fn main() {
    println!("Hello, world!");
}

Also; what do you feel about formatting the template file as well according to how we have formatted the rest of this codebase?

[sivertjoe]

We can do that

[seblyng]

I took my freedom to just update the PR description with a very simple and naive script to help with migration

[sivertjoe]

Idk if I used it incorrectly but ./script 2023 did not work for me 🤪. What do you think of this script

#!/bin/bash

TARGET=$1

cd $TARGET
deps=($(cargo tree -e normal --depth 1 -q | grep -E '├──|└──' | awk '{print $2}'))
for folder in ./*; do
    if [[ $(basename "$folder") == day_* ]]; then
                found=()
                main="${folder}/main.rs"
                for dep in "${deps[@]}"; do
                        if grep -q "$dep" "$main"; then
                                found+=("$dep")
                        fi
                done
                mv $folder temp
                cargo new $folder -q
                cp temp/main.rs "${folder}/src/main.rs"
                cd $folder
                for dep in "${found[@]}"; do
                        cargo add $dep -q
                done
                cd ..
                rm -r temp
        fi
done
rm Cargo.*

[seblyng]

Didn't try it myself yet, but it looks great😎

[sivertjoe]

😎👊

…

________________________________ From: Sebastian Lyng Johansen ***@***.***> Sent: Friday, November 29, 2024 11:43:17 AM To: seblj/cargo-aoc ***@***.***> Cc: Sivert Johansen ***@***.***>; Author ***@***.***> Subject: Re: [seblj/cargo-aoc] Feat/old template (PR #69) Didn't try it myself yet, but it looks great😎 — Reply to this email directly, view it on GitHub<#69 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI5H2YG3RMN7J4RGEEW5HFL2DBAMLAVCNFSM6AAAAABSVJRAWOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBXGU2DKNRYGY>. You are receiving this because you authored the thread.Message ID: ***@***.***>