marvin8/feed2fedi
Watch
1
0
Fork
You've already forked feed2fedi
 0
Code Issues 1 Pull requests Releases 51 Activity Actions

Bot keep yt_dlp temp files in current directory #17

New issue
Closed
opened 2023-12-22 22:54:23 +00:00 by MrClon · 7 comments
MrClon commented 2023-12-22 22:54:23 +00:00 (Migrated from codeberg.org)
Copy link

In some cases Fediverse._post_video download media, but not put in in filenames list, not upload it, and consequently not delete downloaded file. So after some time working directory filled with files like 2000455408_456244066.m4a

In some cases `Fediverse._post_video` download media, but not put in in `filenames` list, not upload it, and consequently not delete downloaded file. So after some time working directory filled with files like `2000455408_456244066.m4a`
marvin8 commented 2023-12-28 20:21:20 +00:00 (Migrated from codeberg.org)
Copy link

I'll have another dive into this. I spent a fair bit of time testing around that but obviously missed an edge case.

I'll have another dive into this. I spent a fair bit of time testing around that but obviously missed an edge case.
marvin8 commented 2024-01-06 05:18:51 +00:00 (Migrated from codeberg.org)
Copy link

I found and corrected another edge case where the temporary files were kept. Hopefully I have now quashed this bug.

I found and corrected another edge case where the temporary files were kept. Hopefully I have now quashed this bug.
marvin8 commented 2024-01-07 01:05:36 +00:00 (Migrated from codeberg.org)
Copy link

@MrClon I think I am ready to publish a new version of Feed2Fedi.

If you have a chance in the next day or two, can you test the wip branch and let me know if you encounter any issues and if this issue has been addressed for you, please?

PS:
Keep in mind there is a breaking change with the reading the config file. See https://codeberg.org/MarvinsMastodonTools/feed2fedi/src/branch/wip/changelog.d/20240107_101406_marvin8_wip.rst for details.

@MrClon I think I am ready to publish a new version of Feed2Fedi. If you have a chance in the next day or two, can you test the `wip` branch and let me know if you encounter any issues and if this issue has been addressed for you, please? PS: Keep in mind there is a breaking change with the reading the config file. See https://codeberg.org/MarvinsMastodonTools/feed2fedi/src/branch/wip/changelog.d/20240107_101406_marvin8_wip.rst for details.
MrClon commented 2024-01-07 01:13:04 +00:00 (Migrated from codeberg.org)
Copy link

Will check it in few days

Will check it in few days
MrClon commented 2024-01-13 13:50:11 +00:00 (Migrated from codeberg.org)
Copy link

Sorry for long waiting. Current wip works well it my setup

Sorry for long waiting. Current `wip` works well it my setup
marvin8 commented 2024-01-14 08:27:36 +00:00 (Migrated from codeberg.org)
Copy link

Thanks... will work releasing WIP branch properly in the next few days.

Thanks... will work releasing WIP branch properly in the next few days.
marvin8 commented 2024-01-17 05:01:07 +00:00 (Migrated from codeberg.org)
Copy link

Addressed in release 2.0.0

Addressed in release 2.0.0
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
pages
4.0.4
4.0.3
4.0.2
4.0.1
4.0.0
3.8.0
3.7.6
3.7.5
3.7.4
3.7.3
3.7.2
3.7.1
3.7.0
3.6.1
3.6.0
3.5.3
3.5.2
3.5.1
3.5.0
3.4.5
3.4.4
3.4.3
3.4.2
commits3.4.2
3.4.1
3.4.0
3.3.2
3.3.1
3.3.0
3.2.12
3.2.11
3.2.10
3.2.9
3.2.8
3.2.7
3.2.6
3.2.5
3.2.4
3.2.3
3.2.2
3.2.1
3.2.0
3.1.12
3.1.1
3.1.0
3.0.0
2.0.0
1.0.0
0.4.1
0.4.0
0.3.3
0.3.2
0.3.1
0.3.0
0.2.1
0.2.0
0.1.0
Labels
Clear labels   
bug
Something is not working

  
contribution welcome
Contributions are very welcome, get started here

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
good first issue
Interested in contributing? Get started here.

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
upstream
Related to an upstream repository, already reported there

No labels
bug
contribution welcome
duplicate
enhancement
good first issue
help wanted
invalid
question
upstream
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/feed2fedi#17
No description provided.