Errors building current repository



  • 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



  • 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.



  • 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



  • 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.



  • 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



  • @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



  • Hello Claudia,

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

    Thank you.



  • 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



  • Hello Claudia,

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

    Could you please see this?

    BR.



  • 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



  • 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



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

    Great!
    Many thanks again.
    BR.



  • Being an amateur… :)



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

    Thx - but you aren’t ;-)

    Cheers
    Claudia



  • 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.



  • @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)
    


  • @dail

    thank you very much for clarifying.

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

    Cheers
    Claudia



  • @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.


Log in to reply