Consider renaming the shared libraries depending on debug/release

[bobhairgrove]

This is such a useful widget ... I hope that it finds a wide audience for applications that need this! 👏

It might be a good thing to do if we had different names of libqhexedit.so depending on whether a library was compiled under debug or release mode, i.e. (for those of us who like to separate these):

build/Debug/libqhexeditd.so
...
build/Release/libqhexedit.so

NOTE: Added "d" to file name for debug version. At least on Windows, this is standard practice, AFAIK; on Linux, people will probably drop the source code into whatever project they are using...

[Pandapip1]

AFAIK; on Linux, people will probably drop the source code into whatever project they are using

This has not been my experience. Generally, most projects do seem to correctly use CMake's find_package, which I believe qhexedit supports.

[Simsys]

There is already a definition in qhexedit.pro that I assumed would do exactly that:

Release:TARGET = qhexedit
Debug:TARGET = qhexeditd

Any idea how this can be implemented?

[bobhairgrove]

There is already a definition in qhexedit.pro that I assumed would do exactly that:

Release:TARGET = qhexedit
Debug:TARGET = qhexeditd

Hmm ... I built both Debug and Release just now, and it seems to work OK! I must have overlooked something before.

Sorry about the noise!

[bobhairgrove]

I downloaded the newest sources now, and it names both Debug and Release the same on Linux! :( I must have renamed the files manually before and forgot about it.

I think your setup probably works on Windows, but I haven't tested it.

(Using Ubuntu 22.04 - Qt 5.15.5 - g++ version 11.4.0)

[bobhairgrove]

I was able to remedy this. It is described in the Qt qmake manual under "Building and Installing in Debug and Release Modes -- Building in Both Modes":

To enable a project to be built in both modes, you must add the debug_and_release option to the CONFIG variable:

 CONFIG += debug_and_release

 CONFIG(debug, debug|release) {
     TARGET = debug_binary
 } else {
     TARGET = release_binary
 }

The scope in the above snippet modifies the build target in each mode to ensure that the resulting targets have different names. Providing different names for targets ensures that one will not overwrite the other.

This is what I have now:

CONFIG += debug_and_release
CONFIG(debug, debug|release) {
    TARGET = qhexeditd
} else {
    TARGET = qhexedit
}

While we are on the subject of qmake, you might want to keep the temporary artifacts of the build in separate directories, for example:

# Keep build artifacts outside of the source tree:
OBJECTS_DIR = ./.obj
MOC_DIR     = ./.moc
RCC_DIR     = ./.qrc
UI_DIR      = ./.ui

But this is just my personal preference.

[bobhairgrove]

Think I'll reopen this because it still needs to be fixed.

[Simsys]

Can you please create a pull request so I can see exactly what you want to change? Thank you.

[bobhairgrove]

Can you please create a pull request so I can see exactly what you want to change? Thank you.

OK ... I will have to fork the repo first. Can I assume that master is the correct branch to use?

[Simsys]

fixed, see #179