Display filename when fxconv raise an error #5

Open
opened 2 years ago by Darks · 4 comments
Darks commented 2 years ago

When compiling with -j*, fxconv is called many times in parallel. Thus, error messages may not be synchronized.

Example below:

fxconv --bopti-image assets-cg/img/sprites/sprite1_back.png -o build-cg/assets/img/sprites/sprite7_back.png.o --cg --toolchain=sh-elf name:img_sprite7_back  profile:p4
...
fxconv --bopti-image assets-cg/img/sprites/sprite8_big.png -o build-cg/assets/img/sprites/sprite8_big.png.o --cg --toolchain=sh-elf name:img_sprite8_big  profile:p4
fxconv --bopti-image assets-cg/img/sprites/sprite8_front.png -o build-cg/assets/img/sprites/sprite8_front.png.o --cg --toolchain=sh-elf name:img_sprite8_front  profile:p4
/usr/lib/python3.8/site-packages/PIL/Image.py:1044: UserWarning: Couldn't allocate palette entry for transparency
  warnings.warn("Couldn't allocate palette entry for transparency")
/usr/lib/python3.8/site-packages/PIL/Image.py:1044: UserWarning: Couldn't allocate palette entry for transparency
  warnings.warn("Couldn't allocate palette entry for transparency")

It may be great to display the filename in the log entry when fxconv raise a warning or an error.

When compiling with `-j*`, `fxconv` is called many times in parallel. Thus, error messages may not be synchronized. Example below: ``` fxconv --bopti-image assets-cg/img/sprites/sprite1_back.png -o build-cg/assets/img/sprites/sprite7_back.png.o --cg --toolchain=sh-elf name:img_sprite7_back profile:p4 ... fxconv --bopti-image assets-cg/img/sprites/sprite8_big.png -o build-cg/assets/img/sprites/sprite8_big.png.o --cg --toolchain=sh-elf name:img_sprite8_big profile:p4 fxconv --bopti-image assets-cg/img/sprites/sprite8_front.png -o build-cg/assets/img/sprites/sprite8_front.png.o --cg --toolchain=sh-elf name:img_sprite8_front profile:p4 /usr/lib/python3.8/site-packages/PIL/Image.py:1044: UserWarning: Couldn't allocate palette entry for transparency warnings.warn("Couldn't allocate palette entry for transparency") /usr/lib/python3.8/site-packages/PIL/Image.py:1044: UserWarning: Couldn't allocate palette entry for transparency warnings.warn("Couldn't allocate palette entry for transparency") ``` It may be great to display the filename in the log entry when fxconv raise a warning or an error.
Owner

That's possible, though the warning you're getting right now is from PIL, not fxconv, so I can't catch it or change it.

It's funny to see how fxconv-like this error message is, but if you look closely it's nowhere in the source code. ^^

I'll check fxconv errors to make sure to add the file path here at least.

That's possible, though the warning you're getting right now is from PIL, not fxconv, so I can't catch it or change it. It's funny to see how fxconv-like this error message is, but if you look closely it's nowhere in the source code. ^^ I'll check fxconv errors to make sure to add the file path here at least.

Hmm, 8 month ago, so this issue is very in a inactive state, you should be close this issue.

// Sundev79 - Thanks

Hmm, 8 month ago, so this issue is very in a inactive state, you should be close this issue. // Sundev79 - Thanks

If anything, I'm the one being inactive - the issue is still there. I see no reason to close.

If anything, I'm the one being inactive - the issue is still there. I see no reason to close.

OK

OK
Sign in to join this conversation.
No Label
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.