feddit.org

2 points

3 months ago

I think Ideally you should be able to run it as you instead of root or sudo. I’m assuming you are needing sudo since you don’t have access to the burner as your user.

Do an “ls -lah” on your cd burner. I think you said it was /dev/sr0 so “ls -lah /dev/sr0” and see if it is owned by root:root or hopefully root:disk or something like that. The format is “user:group” so I’m hoping it is owned by a group that you can simply add your user to.

If it is owned by another group, you can just run “sudo usermod -aG disk user” replace disk with the group that shows on the ls command and user with your user.

If that burner is owned by root:root, there is a way to change that. But that gets very complicated. And I’m not sure its worth the effort for you unless you are wanting to learn more. Point 4.3 here: https://wiki.archlinux.org/title/Udev

In which case to directly answer your question, I’d personally prefer to sudo the script instead of adding sudo in the script. But at the end of the day, I don’t think it matters too much for this specific use case.

report

[ - ]

1 point

3 months ago

Looks like root, I think. It says

brw-rw----+ 1 root cdrom 11, 0 Aug 18 14:13 /dev/sr0

no “:” though oddly, so maybe I can run

sudo usermod -aG cdrom $USER?

Edit: looks like it throws the same error as not sudoing, wodim no write mode specified blah blah. It did add me to the cdrom group though. Although now it won’t work with sudo, how do I remove myself from the cdrom group?

Edit again: Wait, I got it working with another disk, the one I was just trying may have been too big to fit on the disk but throwing the same error as when I didn’t use sudo. Burning this one with sudo, will test again without when it finishes. Thankfully I have a stack of these disks lol I can do this all day.

Ok, still need sudo. Without sudo it just exits without writing to the disk. I guess what I thought was warnings is just standard incomprehensible readout, but yeah without sudo (or if the files are too big for the disk) it just exits and finishes out my script removing the .cdr files.

report

[ - ]

2 points

3 months ago

My bad thought it was printed as “user:group” and not “user group”.

When you add yourself to a group you need to either log out and log back in or reboot in order for it to take effect. So maybe next time you log in try it without sudo again.

report

[ - ]

1 point

3 months ago

Oh cool thanks I’ll log out and try again. Ran into another issue in my script I’m trying to work around now though: I disconnected and reconnected the drive, now it’s /dev/sr1!

So, I guess I need to have my script run cdrecord -checkdrive, and then take that answer as a variable $CDROM and pump it back into cdrecord -v dev=/dev/$CDROM -audio yadda yadda.

This is getting a liiitle above my head lmao.

Edit: logged out and in, no dice, still sudo. Now to figure out this checkdrive issue…

report

[ - ]

simplymath@lemmy.world

2 points

3 months ago

if you want to take a break, I can ask somebody I know who might have a bit more experience with this.

report

[ - ]

1 point

3 months ago

I think I need one! Lol, I’ll happily take a break, thanks for the help and let me know what your friend says!

report

[ - ]

simplymath@lemmy.world

2 points

3 months ago

I messaged him. No guarantees or anything, but he was my forum guy before I was your forum guy.

report

Show more comments

[ - ]

1 point

3 months ago

Ahhh, so apparently in the man page for cdrecord it mentions it needs to be ran as root since it uses “real time scheduling” to write. So even if you have proper permissions to use the cd burner, you still need root to run it. I made a bad assumption that you were having to use root since you didn’t have permissions as your use to write to it.

If you don’t need to parse the output of “cdrecord -checkdrive” then setting that var is pretty trivial.

CDROM=$(cdrecord -checkdrive)

If that outputs more than just the string you need, that gets a little headachey. Grep/awk/sed/sort/uniq/regex are all very powerful and esoteric.

That being said, the man page also mentions that most users will not have to specify “dev” at all as it should figure it out automatically. So you might be ok with axing the “dev” part of the command instead of feeding it the device path.

report

[ - ]

1 point

3 months ago

AH! Oh well no harm no foul, I can run my script as root, I trust me!

Cdrecord -checkdrive does output a whole mess of stuff:

Device was not specified. Trying to find an appropriate drive...
Detected CD-R drive: /dev/sr0
Device type    : Removable CD-ROM
Version        : 0
Response Format: 2
Capabilities   : 
Vendor_info    : 'Slimtype'
Identification : 'eNAU108   8     '
Revision       : 'XL0A'
Device seems to be: Generic mmc2 DVD-R/DVD-RW.
Using generic SCSI-3/mmc   CD-R/CD-RW driver (mmc_cdr).
Driver flags   : MMC-3 SWABAUDIO BURNFREE FORCESPEED 
Supported modes: TAO PACKET SAO SAO/R96P SAO/R96R RAW/R16 RAW/R96P RAW/R96R

And the only part I need is the part that says /dev/sr0/.

I will try deleting the /dev/ part entirely though and see what happens (probably tomorrow evening). If it works that’ll be perfect. Thanks!

Edit: No dice, dev=/dev/sr0 is needed in my case.

report

[ - ]

2 points

3 months ago

This is a very goofy workaround for you that doesn’t actually check what the device is. Only checks if /dev/sr0 exists and if yes, use that if no then use /dev/sr1. Better solution below this block. But leaving it because I kind of like it’s jankiness.

if [ -e /dev/sr0]; then
    DEVICE="/dev/sr0"
else
    DEVICE="/dev/sr1"
fi

Not elegant and a little janky but works.

This will work better. We are taking the output of -checkdrive, searching for “Detected” and sending that to awk. We are telling awk to split that line into columns based on this character “:” and to print the second column. That should give you an output of " /dev/sr0" with a space in front.

DEVICE=$(cdrecord -checkdrive | grep Detected | awk -F ":" '{print $2}')

That should work. But if you absolutely must kill the whitespace for some reason we can add trim to the end like so

DEVICE=$(cdrecord -checkdrive | grep Detected | awk -F ":" '{print $2}' | tr -d ' ')

There might be a more elegant solution by using the output of “cdrecord -scanbus” instead. No clue though since I don’t have the hardware to verify from here. Hope that helps!

report