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

add constants used by alternate saver of patchelf.rb #44

Merged
merged 1 commit into from
Aug 21, 2020

Conversation

rmNULL
Copy link
Contributor

@rmNULL rmNULL commented Aug 19, 2020

constants required for david942j/patchelf.rb#28

@david942j
Copy link
Owner

Please rebase, I just fixed the CI issues.

Copy link
Owner

@david942j david942j left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@@ -95,6 +95,7 @@ module DT
DT_VERDEF = 0x6ffffffc # address of version definition table
DT_VERDEFNUM = 0x6ffffffd # number of entries in {DT_VERDEF}
DT_VERNEED = 0x6ffffffe # address of version dependency table
DT_VERSYM = 0x6ffffff0 # section address of .gnu.version.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the period in the end

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -212,6 +213,15 @@ def self.mapping(type)
end
include ET

# segment permission flags, bitwise ored value is recorded in +p_flags+
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make it a sentence (i.e. capitalize, add period)
Same below

These constants are used by alternate saver of patchelf.rb
@rmNULL rmNULL force-pushed the master branch 3 times, most recently from a99bdb9 to e897f8d Compare August 21, 2020 07:54
@rmNULL rmNULL requested a review from david942j August 21, 2020 08:19
@rmNULL
Copy link
Contributor Author

rmNULL commented Aug 21, 2020

good now ?

@david942j david942j merged commit 2c3c652 into david942j:master Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants