libwiisprite on github

Post Reply
nebiun
Posts: 48
Joined: Sat Dec 28, 2013 2:42 pm

libwiisprite on github

Post by nebiun » Sun Dec 23, 2018 2:17 pm

I merged source files and examples, modified the makefile to compile with the latest devkitPPC/libogc and then pushed all files on github.

https://github.com/nebiun/libwiisprite

WinterMute
Site Admin
Posts: 1403
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: libwiisprite on github

Post by WinterMute » Sun Dec 23, 2018 6:51 pm

If you're going to make this kind of effort (which, btw is appreciated, don't get me wrong here) it would be good if things were updated for modern standards.

Several files in there just contain the output of raw2c rather than having them generated at build time which makes it awkward to modify and encourages other people to do the same.

Loading png files at runtime & hardcoding references to them is a terrible way to manage assets too. I'd really like to see more in the way of asset management tools around (preferably cross platform). We have tex3ds for generating 3ds texture atlases which could, in theory, be expanded to deal with wii/gamecube textures too.

The habit people have of just dumping all their libraries into the toolchain installation without bothering to talk to anyone needs to stop as well. It causes major headaches with people ending up with incompatible things and various conflicts. We really want things contributed to https://github.com/devkitPro/pacman-packages rather than having a load of random things all over the place setting traps for novices.

Digging through all this I see absolute paths all over the place when data files should just be in the same place as the dol & loaded with relative paths.

It's good to see people around making the effort to update the libraries they're using for latest tools though. It really is appreciated. I'd like to try and pull everything together more so anyone trying to get started with all this has more of a chance of succeeding.
Help keep devkitPro toolchains free, Donate today

devkitPro IRC support
Personal Blog

nebiun
Posts: 48
Joined: Sat Dec 28, 2013 2:42 pm

Re: libwiisprite on github

Post by nebiun » Mon Dec 24, 2018 10:46 am

WinterMute wrote:If you're going to make this kind of effort (which, btw is appreciated, don't get me wrong here) it would be good if things were updated for modern standards.

Several files in there just contain the output of raw2c rather than having them generated at build time which makes it awkward to modify and encourages other people to do the same.

Loading png files at runtime & hardcoding references to them is a terrible way to manage assets too. I'd really like to see more in the way of asset management tools around (preferably cross platform). We have tex3ds for generating 3ds texture atlases which could, in theory, be expanded to deal with wii/gamecube textures too.

The habit people have of just dumping all their libraries into the toolchain installation without bothering to talk to anyone needs to stop as well. It causes major headaches with people ending up with incompatible things and various conflicts. We really want things contributed to https://github.com/devkitPro/pacman-packages rather than having a load of random things all over the place setting traps for novices.

Digging through all this I see absolute paths all over the place when data files should just be in the same place as the dol & loaded with relative paths.

It's good to see people around making the effort to update the libraries they're using for latest tools though. It really is appreciated. I'd like to try and pull everything together more so anyone trying to get started with all this has more of a chance of succeeding.
Well, WinterMute.
Give me a guide line and I do it for this library. :wink:

nebiun
Posts: 48
Joined: Sat Dec 28, 2013 2:42 pm

Re: libwiisprite on github

Post by nebiun » Thu Dec 27, 2018 4:14 pm

WinterMute wrote:Several files in there just contain the output of raw2c rather than having them generated at build time which makes it awkward to modify and encourages other people to do the same.
Done! Now binary files are in data directory and Makefile was modified to do the dirty work.
WinterMute wrote:Loading png files at runtime & hardcoding references to them is a terrible way to manage assets too
I leave this for an example because the intention of the programmer was to show the two metods to load images.
WinterMute wrote:The habit people have of just dumping all their libraries into the toolchain installation without bothering to talk to anyone needs to stop as well
Correct. What do you thing about add:
export USERLIBS := $(DEVKITPATH)/userlibs/ppc
to base_tools? If a library is not related with the toolchain and is not a ported library, then it must be installed in USERLIBS ((libgrr, libml, libwiisprite and so on).

WinterMute
Site Admin
Posts: 1403
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: libwiisprite on github

Post by WinterMute » Wed Feb 13, 2019 9:10 pm

nebiun wrote:
Thu Dec 27, 2018 4:14 pm
WinterMute wrote:The habit people have of just dumping all their libraries into the toolchain installation without bothering to talk to anyone needs to stop as well
Correct. What do you thing about add:
export USERLIBS := $(DEVKITPATH)/userlibs/ppc
to base_tools? If a library is not related with the toolchain and is not a ported library, then it must be installed in USERLIBS ((libgrr, libml, libwiisprite and so on).
I'd prefer that nothing that hasn't been officially adopted by devkitPro is placed inside the devkitpro folder at all.

Here's one of the many reasons why.

https://github.com/GRRLIB/GRRLIB/blob/m ... #L165-L188
Help keep devkitPro toolchains free, Donate today

devkitPro IRC support
Personal Blog

nebiun
Posts: 48
Joined: Sat Dec 28, 2013 2:42 pm

Re: libwiisprite on github

Post by nebiun » Wed Feb 13, 2019 9:46 pm

WinterMute wrote:
Wed Feb 13, 2019 9:10 pm
nebiun wrote:
Thu Dec 27, 2018 4:14 pm
WinterMute wrote:The habit people have of just dumping all their libraries into the toolchain installation without bothering to talk to anyone needs to stop as well
Correct. What do you thing about add:
export USERLIBS := $(DEVKITPATH)/userlibs/ppc
to base_tools? If a library is not related with the toolchain and is not a ported library, then it must be installed in USERLIBS ((libgrr, libml, libwiisprite and so on).
I'd prefer that nothing that hasn't been officially adopted by devkitPro is placed inside the devkitpro folder at all.

Here's one of the many reasons why.

https://github.com/GRRLIB/GRRLIB/blob/m ... #L165-L188
My version of grrlib uses only devkitpro (portlibs) libraries, not a local version of them.

WinterMute
Site Admin
Posts: 1403
Joined: Tue Aug 09, 2005 3:21 am
Location: UK
Contact:

Re: libwiisprite on github

Post by WinterMute » Wed Feb 13, 2019 10:08 pm

nebiun wrote:
Wed Feb 13, 2019 9:46 pm
WinterMute wrote:
Wed Feb 13, 2019 9:10 pm
I'd prefer that nothing that hasn't been officially adopted by devkitPro is placed inside the devkitpro folder at all.

Here's one of the many reasons why.

https://github.com/GRRLIB/GRRLIB/blob/m ... #L165-L188
My version of grrlib uses only devkitpro (portlibs) libraries, not a local version of them.
That's not what I'm pointing out.
Help keep devkitPro toolchains free, Donate today

devkitPro IRC support
Personal Blog

Post Reply

Who is online

Users browsing this forum: No registered users and 0 guests