Skip to content

reload_lib -a: [Bug fix] Compares against default branch#21382

Open
g0tmi1k wants to merge 1 commit intorapid7:masterfrom
g0tmi1k:reload_all
Open

reload_lib -a: [Bug fix] Compares against default branch#21382
g0tmi1k wants to merge 1 commit intorapid7:masterfrom
g0tmi1k:reload_all

Conversation

@g0tmi1k
Copy link
Copy Markdown
Contributor

@g0tmi1k g0tmi1k commented Apr 27, 2026

There was a bug with reload_lib -a not working.

Note: master is the default branch for this git repo.

Before

$ git checkout master
[...]
$ ./msfconsole -q
msf > reload_lib
Usage: reload_lib lib/to/reload.rb [...]

Reload Ruby library files from specified paths.

    -h, --help                       Help banner.
    -a, --all                        Reload all* changed files in your current Git working tree.
                                     *Excludes modules and non-Ruby files.
msf > reload_lib -h
Usage: reload_lib lib/to/reload.rb [...]

Reload Ruby library files from specified paths.

    -h, --help                       Help banner.
    -a, --all                        Reload all* changed files in your current Git working tree.
                                     *Excludes modules and non-Ruby files.
msf > reload_lib -a
Usage: reload_lib lib/to/reload.rb [...]

Reload Ruby library files from specified paths.

    -h, --help                       Help banner.
    -a, --all                        Reload all* changed files in your current Git working tree.
                                     *Excludes modules and non-Ruby files.
msf >

After

$ git checkout  reload_all
[...]
$ ./msfconsole -q
msf > reload_lib -a
[*] Reloading /usr/share/metasploit-framework2/lib/msf/ui/console/command_dispatcher/developer.rb
msf >

@g0tmi1k g0tmi1k force-pushed the reload_all branch 2 times, most recently from 9a7756c to 73cba8f Compare April 30, 2026 11:58
@g0tmi1k g0tmi1k changed the title reload_lib: Compares against master branch reload_lib -a: [Bug fix] Compares against default branch May 4, 2026
@g0tmi1k g0tmi1k force-pushed the reload_all branch 3 times, most recently from 55d0293 to d4b5e50 Compare May 6, 2026 16:28
@adfoster-r7
Copy link
Copy Markdown
Contributor

Looks like this would break the reload_lib functionality for my env, since my origin/master is never in sync with upstream/master 👀

Do we want to add a config option or something similar to support this workflow maybe, to avoid potentially breaking for other folk's envs? 🤔

@g0tmi1k
Copy link
Copy Markdown
Contributor Author

g0tmi1k commented May 7, 2026

@adfoster-r7 - Good point! Didn't think about that.
Following: https://docs.metasploit.com/docs/development/get-started/setting-up-a-metasploit-development-environment.html

It says about adding a upstream remote - so it will now attempt to use that first.

Not sure if its on the cards from merging form master to main as the default branch, but incase it is, Ive added that in now...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

3 participants