Skip to content

Commit 95b2a38

Browse files
committed
cmd/evm: add back removed tracing flags and note that they are deprecated
1 parent a0a8603 commit 95b2a38

File tree

3 files changed

+18
-2
lines changed

3 files changed

+18
-2
lines changed

cmd/evm/internal/t8ntool/flags.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,10 @@ var (
3030
Name: "trace",
3131
Usage: "Output full trace logs to files <txhash>.jsonl",
3232
}
33+
TraceDisableMemoryFlag = cli.BoolTFlag{
34+
Name: "trace.nomemory",
35+
Usage: "Disable full memory dump in traces (deprecated)",
36+
}
3337
TraceEnableMemoryFlag = cli.BoolFlag{
3438
Name: "trace.memory",
3539
Usage: "Enable full memory dump in traces",
@@ -38,6 +42,10 @@ var (
3842
Name: "trace.nostack",
3943
Usage: "Disable stack output in traces",
4044
}
45+
TraceDisableReturnDataFlag = cli.BoolTFlag{
46+
Name: "trace.noreturndata",
47+
Usage: "Disable return data output in traces (deprecated)",
48+
}
4149
TraceEnableReturnDataFlag = cli.BoolFlag{
4250
Name: "trace.returndata",
4351
Usage: "Enable return data output in traces",

cmd/evm/internal/t8ntool/transition.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,11 +99,17 @@ func Transition(ctx *cli.Context) error {
9999
return NewError(ErrorIO, fmt.Errorf("failed creating output basedir: %v", err))
100100
}
101101
if ctx.Bool(TraceFlag.Name) {
102+
if ctx.IsSet(TraceDisableMemoryFlag.Name) && ctx.IsSet(TraceEnableMemoryFlag.Name) {
103+
return NewError(ErrorConfig, fmt.Errorf("can't use both flags --%s and --%s", TraceDisableMemoryFlag.Name, TraceEnableMemoryFlag.Name))
104+
}
105+
if ctx.IsSet(TraceDisableReturnDataFlag.Name) && ctx.IsSet(TraceEnableReturnDataFlag.Name) {
106+
return NewError(ErrorConfig, fmt.Errorf("can't use both flags --%s and --%s", TraceDisableReturnDataFlag.Name, TraceEnableReturnDataFlag.Name))
107+
}
102108
// Configure the EVM logger
103109
logConfig := &vm.LogConfig{
104110
DisableStack: ctx.Bool(TraceDisableStackFlag.Name),
105-
EnableMemory: ctx.Bool(TraceEnableMemoryFlag.Name),
106-
EnableReturnData: ctx.Bool(TraceEnableReturnDataFlag.Name),
111+
EnableMemory: !ctx.Bool(TraceDisableMemoryFlag.Name) || ctx.Bool(TraceEnableMemoryFlag.Name),
112+
EnableReturnData: !ctx.Bool(TraceDisableReturnDataFlag.Name) || ctx.Bool(TraceEnableReturnDataFlag.Name),
107113
Debug: true,
108114
}
109115
var prevFile *os.File

cmd/evm/main.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,10 @@ var stateTransitionCommand = cli.Command{
138138
Action: t8ntool.Transition,
139139
Flags: []cli.Flag{
140140
t8ntool.TraceFlag,
141+
t8ntool.TraceDisableMemoryFlag,
141142
t8ntool.TraceEnableMemoryFlag,
142143
t8ntool.TraceDisableStackFlag,
144+
t8ntool.TraceDisableReturnDataFlag,
143145
t8ntool.TraceEnableReturnDataFlag,
144146
t8ntool.OutputBasedir,
145147
t8ntool.OutputAllocFlag,

0 commit comments

Comments
 (0)