NSIS 3

Bug #1620100 reported by poy
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DC++
Fix Released
Low
Unassigned

Bug Description

Tags: installer nsis
poy (poy)
Changed in dcplusplus:
importance: Undecided → Low
Revision history for this message
eMTee (realprogger) wrote :

The current installer script builds and runs fine with NSIS 3.0 so I guess it's best to support both for the time being.

Changed in dcplusplus:
status: Confirmed → Fix Committed
Revision history for this message
maksis (maksis) wrote :

I'd be more strict with the allowed NSIS version: https://textslashplain.com/2015/12/18/dll-hijacking-just-wont-die/

Revision history for this message
poy (poy) wrote :

ouch! yeah, better be up-to-date then...

eMTee, I notice you don't set the "unicode" flag <http://nsis.sourceforge.net/Docs/Chapter4.html#aunicodetarget> to "true" in the commit?

Revision history for this message
eMTee (realprogger) wrote :

poy, nope. Should have I? Compared to the old NSIS Unicode engine I don't see any difference in the way of look and work of the build with NSIS 3.0, using the committed script...

Revision history for this message
eMTee (realprogger) wrote :

I added "Unicode true" which now processes all nsh files and makes a multi language installer, though references of Hindi language must be removed from i18n.nsh to get an error free build.
Apparently the required files Hindi.nlf/nsh are missing from NSIS 3.0's Contrib\Language files\ folder.

Revision history for this message
eMTee (realprogger) wrote :

It's now NSIS 3.0 only and since the DC++ distro build script automatically checks for the available NSIS language files, missing languages in NSIS 3.0 should not be a problem when building the distro.

Revision history for this message
poy (poy) wrote :

that is annoying but not terribly important; adapted the script to be clearer about missing languages and created bug #1623625.

Revision history for this message
poy (poy) wrote :

Fixed in DC++ 0.860.

Changed in dcplusplus:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.