Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

manifest always applied to resourcename [1] #49

Open
tumagonx opened this issue Jul 16, 2017 · 6 comments
Open

manifest always applied to resourcename [1] #49

tumagonx opened this issue Jul 16, 2017 · 6 comments
Labels

Comments

@tumagonx
Copy link

As according to MS Manifest Tool, dll should be MAKEINTRESOURCEW(2) and only exe that use MAKEINTRESOURCEW(1)

rescle.cc :

    (ru.Get()
      , RT_MANIFEST
      , MAKEINTRESOURCEW(1)
      , 1033 // this is hardcoded at 1033, ie, en-us, as that is what RT_MANIFEST default uses
      , &stringSection.at(0), sizeof(char) * stringSection.size())) {

      return false;
    }
@malept
Copy link
Member

malept commented Jul 16, 2017

If editing DLLs is within the scope of rcedit, perhaps we need a --dll flag?

@tumagonx
Copy link
Author

tumagonx commented Jul 16, 2017

run rcedit

Fatal error: You should specify a exe/dll file

yes DLLs is within the scope :)
and no not --dll but https://msdn.microsoft.com/en-us/library/ms809762.aspx
in Characteristics flag 0x2000 means it's dll

@malept
Copy link
Member

malept commented Jul 16, 2017

I ask because the description of the repository is

Command line tool to edit resources of exe

@tumagonx
Copy link
Author

well, a dll is 99,99% an exe too, why not? and because of that --dll (by user) will cause inaccuracy, just my two cents. It's up to the developer...

@malept
Copy link
Member

malept commented Jul 17, 2017

Speaking for myself, feel free to file a pull request so that it autodetects whether it's a DLL or EXE and sets the appropriate magic number.

@tumagonx
Copy link
Author

sorry i made lame C-ish patch barely based on that msdn article
rescle-manifest-dll.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants