-
Notifications
You must be signed in to change notification settings - Fork 55
GSoC '25 Community Bonding and First Week report by Krish #174
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
Conversation
Sorry if I went a little overboard writing, it was fun! |
sudo rm -rf /usr/lib/python3.13/site-packages/sugar3;sudo cp -r ./src/sugar3 /usr/lib/python3.13/site-packages/sugar3 | ||
|
||
# Complete rebuild and install cycle | ||
sudo ./autogen.sh;sudo make clean;sudo make;sudo make install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The copying is usually done after the build, I'm wondering why you're doing it before in this case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just 2 seperate shell commands, I am just mentioning that I clubbed them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see that they're different commands, there's no mention of you clubbing them — what does clubbing mean?.
I'm fine with your blog report detailing your workflow, we won't need to change any of our docs to fit this when the time comes so it's cool.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When chaining commands like this, it can be better to make a compound command linked by &&
than using ;
, e.g. make && make install
That way when one of them fails, the others are not run.
Alternatively, place at the start of your script #!/bin/bash -e
and don't use source
or .
to run it, but chmod +x
and execute it directly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right that gave me an idea to add a binding on these and I can add a shell script.
Thank you!! I will add this on the blogs by next week
@pikurasa kindly review. |
```c | ||
typedef struct { | ||
double x; | ||
double y; | ||
} SugarPoint; | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this appear as expected when rendered?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not really but it' okay :p
Description
Week 1 updates.
Checklist
Checked on browser, seem to work!