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

Using \ to make newlines doesn't work #19

Open
Tracked by #47
Moosems opened this issue May 25, 2023 · 21 comments · May be fixed by #47
Open
Tracked by #47

Using \ to make newlines doesn't work #19

Moosems opened this issue May 25, 2023 · 21 comments · May be fixed by #47
Labels
enhancement New feature or request improve Improvements

Comments

@Moosems
Copy link
Collaborator

Moosems commented May 25, 2023

No description provided.

@littlewhitecloud
Copy link
Owner

?

@Moosems
Copy link
Collaborator Author

Moosems commented May 26, 2023

In a terminal, you can use backslashes to give multiple lines for one big command.

@littlewhitecloud littlewhitecloud added enhancement New feature or request improve Improvements labels May 26, 2023
@littlewhitecloud
Copy link
Owner

@Moosems can you give a screenshot? Or can you assign yourself to improve this function? I really can't image how to use it.

@littlewhitecloud
Copy link
Owner

Is it just like:

cl /Ox /Ot /favor:blend /EHsc \
/GA /nologo /LD /DEF: plugin.def plugin.c \
/w /errorReport:none /std:c17 /D "_WINDOWS"

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 9, 2023

Yeah. In windows you use && but on many other systems and with bash you use \. I'll make an implementation sometime. I'm busy this week though so I can't do nearly and programming right now.

@littlewhitecloud
Copy link
Owner

fixed it in #31

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 11, 2023

Also needs && for windows ;).

@littlewhitecloud
Copy link
Owner

Also needs && for windows ;).

Added it! :D

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 12, 2023

You're epic.

@littlewhitecloud
Copy link
Owner

littlewhitecloud commented Jun 12, 2023

You're epic.

Thanks! Actally it is quite easy to add "&&"!
This is my first time to work with others on my repo. I feel very accomplished. It is meaningful for me to work with a funny guy like you. You are epic too!

@littlewhitecloud
Copy link
Owner

So, what's wrong with the latest code?

@littlewhitecloud
Copy link
Owner

image
I tested a lot but found nothing

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 23, 2023

The backslash isn't removed so the terminal treats the chars as escaped chars.

@littlewhitecloud
Copy link
Owner

littlewhitecloud commented Jun 23, 2023

image
What? It doesn't happen.

@littlewhitecloud
Copy link
Owner

littlewhitecloud commented Jun 27, 2023

Can you give a screenshot or an example? @Moosems
This issues is the last issues in the pr we didn't complete

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 27, 2023

I'm not with my computer this week but from memory here's what happens:

Moosems$ cd \
.
Unable to change to directory (insert escaped period char)

@littlewhitecloud
Copy link
Owner

littlewhitecloud commented Jun 27, 2023

image
nothing happened...?

@littlewhitecloud
Copy link
Owner

quite strange

@littlewhitecloud
Copy link
Owner

littlewhitecloud commented Jun 27, 2023

I'm not with my computer this week but from memory here's what happens:

Moosems$ cd \
.
Unable to change to directory (insert escaped period char)

It can't be, I wrote code:

self.longcmd += cmd.split(self.longsymbol)[0]

to cut off the \\

@littlewhitecloud
Copy link
Owner

Still keep open it.
And try to fix it when @Moosems you have computer and you are free to test.

@Moosems
Copy link
Collaborator Author

Moosems commented Jun 27, 2023

Will do!

@littlewhitecloud littlewhitecloud linked a pull request Jun 30, 2023 that will close this issue
11 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request improve Improvements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants