Skip to content

Conversation

@aquiffoo
Copy link
Contributor

added "os" library ⬇️

I decided to add an "os" library because running commands is essential.
for now it only includes os.exec(command), that runs said command.

@aquiffoo
Copy link
Contributor Author

shit, what's going wrong?

@aquiffoo
Copy link
Contributor Author

oooh... i see! i made the changes on windows, and linux does not support the library i use in os.exec. fixing that asap

@coolbeans1212
Copy link

i believe in you aquiffoo

@aquiffoo
Copy link
Contributor Author

aquiffoo commented Jan 1, 2025

i've made the changes @cyteon. hopefully it becomes compatible with unix-like systems now

Copy link
Contributor Author

@aquiffoo aquiffoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing the buginess of the gh app

@cyteon
Copy link
Owner

cyteon commented Jan 2, 2025

   Compiling modu v0.4.0 (/var/home/cyteon/Desktop/SideProjects/modu/lang)
error[E0433]: failed to resolve: could not find `windows` in `os`
   --> src/packages/os.rs:4:14
    |
4   | use std::os::windows::process::CommandExt;
    |              ^^^^^^^ could not find `windows` in `os`

@cyteon
Copy link
Owner

cyteon commented Jan 2, 2025

I'll merge and fix it myself

@cyteon cyteon merged commit 4370fa1 into cyteon:main Jan 2, 2025
@aquiffoo
Copy link
Contributor Author

aquiffoo commented Jan 2, 2025

ok, tysm

@mustafaC0der
Copy link

Lovely. Rn im giving exams but will try as soon as possible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants