Errors building current repository

Yaron

Hello Claudia and dail,

@Claudia-Frank,

Thank you so much! I really appreciate the detailed reply.

I’ve decided to try the lighter option first.
I’ve installed MagicISO (thanks for that too), and then wdk7.1.

Sorry to bother you yet again.
Could you please elaborate on modifying the include directories?
Where should I do that?

@dail,
Thanks again. I really appreciate your help.

Best regards.

Claudia Frank

Hello Yaron,

either you edit the notepadPlus.vcxproj file directly or you use visual studio to do so.

File editing:
There are two nodes Unicode Debug|Win32 and Unicode Release|Win32

IncludePath needs to be extended with D:\WinDDK\7600.16385.1\inc\atl71
and
LibraryPath with D:\WinDDK\7600.16385.1\lib\ATL\i386

Like here

<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Unicode Debug|Win32'">
	<OutDir>$(Configuration)\</OutDir>
	<IntDir>$(Configuration)\</IntDir>
	<LinkIncremental>false</LinkIncremental>
	<IncludePath>D:\WinDDK\7600.16385.1\inc\atl71;$(IncludePath)</IncludePath>
	<LibraryPath>D:\WinDDK\7600.16385.1\lib\ATL\i386;$(LibraryPath)</LibraryPath>
</PropertyGroup>

<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Unicode Release|Win32'">
	<OutDir>..\bin\</OutDir>
	<IntDir>$(Configuration)\</IntDir>
	<LinkIncremental>false</LinkIncremental>
	<IncludePath>D:\WinDDK\7600.16385.1\inc\atl71;$(IncludePath)</IncludePath>
	<LibraryPath>D:\WinDDK\7600.16385.1\lib\ATL\i386;$(LibraryPath)</LibraryPath>
</PropertyGroup>

when using visual studio, load the solution notepadPlus.sln.
Once done, right click on notepad++ in class view and choose properties.
In properties window select vc++ directories
Click on include directories and a trianlge marker appears on the right,
click it -> two new lines appear click on edit
Add D:\WinDDK\7600.16385.1\inc\atl71
click on apply

Do the same steps for library but add D:\WinDDK\7600.16385.1\lib\ATL\i386

close.

Right click on notepad++ and choose build (or via menu)
No error message should appear.

Cheers
Claudia

Yaron

Hello Claudia,

Thank you very much. I appreciate your kindness and patience.

No errors but multiple warnings.
Should I have changed any thing here?

Best regards.

Claudia Frank

Hello Yaron,

actually I do get the same warnings.
As I use this npp on my pc only ( others do get the official ones ;-)
I ignore them, in order to get rid of the warnings I assume you need to change
alt headers as the error suggest that different permissions are merged.

Hmm … will try to check this out.

Cheers
Claudia

Yaron

Hello Claudia,

If you ignore them, I’ll do the same. :)
And, obviously :), I don’t quite understand your suggestion.
I’ll wait for your further tests.

Many thanks again. I’m truly grateful.

BR.

Claudia Frank

Hello Yaron,

when right clicking on notepad++ in project view
you see a folder external dependencies and under this folder
there is a file called atlbase.h If you double click on it
search for #pragma section(“ATL$__a”, read, shared)
comment the three lines and/or replace it with the ones here.

//#pragma section("ATL$__a", read, shared)
//#pragma section("ATL$__z", read, shared)
//#pragma section("ATL$__m", read, shared)
#pragma section("ATL$__a", read)
#pragma section("ATL$__z", read)
#pragma section("ATL$__m", read)

Warnings gone, basically, what ms says, shared means that all process can get access this section
but from my understanding this isn’t possible within an win32 environment at all due to memory protection
between processes.

So from my point it is even safe to ignore it. (but I did the change - less to read on the output ;-))

Cheers
Claudia

Claudia Frank

@dail,

could you check if the atlbase.h version provided by community edition has still the shared flag?
Actually I don’t see any reason for it. Maybe someone can enlight me.

Cheers
Claudia

Yaron

Hello Claudia,

Fast and furious. :)
Commenting those lines generated some errors.
Replacing them works well (no warnings).

Thank you.

Claudia Frank

Hello Yaron

Fast and furious. :)

In memories of Paul Walker :-(

Commenting those lines generated some errors.

Not for me, I actually commented them

Replacing them works well (no warnings).

Hmmm, sure you commented them correctly ;-)

Cheers
Claudia

Yaron

Hello Claudia,

I have actually not seen the series. I usually prefer European films. :)
Anyway, may he RIP.

Could you please see this?

BR.

Claudia Frank

Hello Yaron,

sorry, I 'm a fool :-D (can’t stop laughing)
What I meant was commenting the lines to still have the old version
but to add the new statements with read flag only.
Basically what I have posted but not said :-D

Sorry for confusion.

Cheers
Claudia

Claudia Frank

Oh nnoo nooo I said it

comment the three lines and/or replace it with the ones here.

comment the three lines and replace it with the ones here.

comment the three lines or replace it with the ones here.

Or?? :-D

Cheers
Claudia

Yaron

You’re certainly no fool. :)
Being an armature, I understood “and/or” literally.

Great!
Many thanks again.
BR.

Yaron

Being an amateur… :)

Claudia Frank

Just used different translator to find out what this means ;-)

Thx - but you aren’t ;-)

Cheers
Claudia

Yaron

Actually the Penny has dropped for me after posting (i.e. I started analyzing the possible meanings too late). :)

Thanks. Everything is relative.
I certainly didn’t know what I was doing here. :)

BR.

dail

@Claudia-Frank said:

could you check if the atlbase.h version provided by community edition has still the shared flag?
Actually I don’t see any reason for it. Maybe someone can enlight me.

Cheers
Claudia

My header does not have the shared flag, it only has

#pragma section("ATL$__a", read)
#pragma section("ATL$__z", read)
#pragma section("ATL$__m", read)

Claudia Frank

@dail

thank you very much for clarifying.

@Yaron,
so we did what MS already did in a newer version of atl.

Cheers
Claudia

Yaron

@dail,

Thank you very much.
Am I not right in officially adding “Sherlock Holmes” to Claudia’s name?
You must be impressed as well, aren’t you?

BR.